Skip to content

feat: integration test suite #51

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
erunion merged 29 commits into from
Sep 14, 2021
Merged

feat: integration test suite #51

Changes from 1 commit
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
b2e6e25
feat: initial pass at rewriting the integration testing framework
erunion Sep 10, 2021
02ab8fd
test: docker tweaks
erunion Sep 10, 2021
75daa15
Merge branch 'main' into feat/integration-tests
erunion Sep 10, 2021
a06d992
style: fixing linting problems
erunion Sep 10, 2021
2d0c160
ci: using optional chaining in tests so don't run them on node 12
erunion Sep 10, 2021
51269a1
feat: first pass at containerizing node integrations
erunion Sep 11, 2021
6ff7e4e
feat: making our useObjectBody option the default behavior in node-fetch
erunion Sep 13, 2021
b4bd8a5
fix: removing multipart/form-data headers from node-fetch when we use…
erunion Sep 13, 2021
4a61242
fix: bug where node request wasn't sending cookies properly
erunion Sep 13, 2021
99faa7d
fix: adding useQuerystring to the node request opts to fix query para…
erunion Sep 13, 2021
7b4acc1
fix: send the full url instead of `qs` for node request because of do…
erunion Sep 13, 2021
f117097
fix: issues with node axios not sending encoded and query params right
erunion Sep 13, 2021
2168051
fix: fixes to the integration tests suite
erunion Sep 13, 2021
a29ab1d
fix: promoting the js+fetch useObjectBody option to be the default be…
erunion Sep 13, 2021
18a7b4d
fix: adding the composer autoloader to the top of guzzle snippets
erunion Sep 13, 2021
00a4ce5
fix: catch and assert errors when we might be hitting a PROPFIND method
erunion Sep 13, 2021
cef2c13
docs: updating our differences list in the readme
erunion Sep 13, 2021
d0a43f3
chore: renaming the `http` fixture to `http-insecure`
erunion Sep 13, 2021
48133d2
test: setting up the integration tests to run on node, php, and python
erunion Sep 13, 2021
fd95dee
feat: getting the docker container running node tests
erunion Sep 13, 2021
4dcc967
ci: workflow for running node integrations
erunion Sep 13, 2021
4d93815
ci: moving integration dockerfiles into their own dir
erunion Sep 13, 2021
1fecafc
ci: containerizing php integrations
erunion Sep 14, 2021
e399acf
ci: no composer files exist to copy
erunion Sep 14, 2021
c7f141d
ci: installing npm deps in the php container so we can run tests
erunion Sep 14, 2021
480c1de
fix: changing python requests target to use full urls instead of supp…
erunion Sep 14, 2021
d7b9da2
ci: containerizing python integration tests
erunion Sep 14, 2021
cb14ccd
ci: containerizing the curl test suite
erunion Sep 14, 2021
533274d
ci: adding a github workflow for the shell integration suite
erunion Sep 14, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
ci: installing npm deps in the php container so we can run tests
  • Loading branch information
@erunion
erunion committed Sep 14, 2021
commit c7f141d86240c281469f993b1b0e8dbd14728aaf
6 changes: 4 additions & 2 deletions integrations/php.Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,12 @@ RUN composer require guzzlehttp/guzzle

FROM alpine

ADD . /src
WORKDIR /src

RUN apk update
RUN apk add php7 php7-fpm php7-opcache php7-curl
RUN apk add --update nodejs npm
RUN npm install

ADD . /src
WORKDIR /src
COPY --from=builder /composer/vendor /src/vendor