Commit Graph

201 Commits

Author SHA1 Message Date
Dimitrie Stefanescu 6cdeccb74b feat(apps): more service implementations, cleanup, and tests 2020-09-18 23:12:43 +01:00
Dimitrie Stefanescu 03bd49ef6d feat(apps): scaffolded more app services & more (cleanup, tests, etc.) 2020-09-18 16:46:23 +01:00
Dimitrie Stefanescu c096ba89cc feat(models): removed username field form user
in dev environmnets, this will require a manual db reset (migrate:rollback, latest)
2020-09-16 10:30:34 +01:00
Dimitrie Stefanescu 3e3d99b782 feat(tracing): improved GQL ops tracing logs 2020-09-03 17:12:57 +01:00
Dimitrie Stefanescu 107266b901 feat(local auth): in dev/test mode, returns a token on the authorization finalisation middleware 2020-09-02 14:04:29 +01:00
Dimitrie Stefanescu 1b5b13bf9d refactor(subs names): renamed streamCreated/Deleted to Added/Removed 2020-09-01 14:37:52 +01:00
Dimitrie Stefanescu 33fb0fa7f7 docs(subs): updates schema comments to clarify userStreamCreated/Deleted re perms granted/revoked 2020-09-01 12:16:24 +01:00
Dimitrie Stefanescu 1f8ef39831 test(subs): uncommented all tests 2020-09-01 12:09:56 +01:00
Dimitrie Stefanescu f24dd6529e test(subs): fixed borked streamUpdate 2020-09-01 12:07:53 +01:00
Dimitrie Stefanescu 28b8c08aff perf(subs): standardises the response on stream subs to id (vs mixture of streamId and id) 2020-09-01 11:58:09 +01:00
Dimitrie Stefanescu 690d95bfde fix(subs): replaces permission subs with created/deleted + sharedBy/revokedBy props 2020-09-01 11:46:38 +01:00
Dimitrie Stefanescu 11571a1d76 fix(subscriptions): added stream deleted subscription 2020-09-01 10:58:29 +01:00
Dimitrie Stefanescu 8a023d25d5 fix(setup): fully removes the setup check app 2020-08-31 18:15:18 +01:00
Dimitrie Stefanescu 16f3e1bb03 fix(subs): wrapps onConnection in a try catch and checks for auth headers
some clients send the connection params with lowercase :/
2020-08-31 13:27:55 +01:00
Dimitrie Stefanescu 0e98b6f5ce feat(subs): adds 'grantor' userid on stream permissions granted event
so later down the line we can do 'x shared y with you', albeit i'm also inclined to move those
permission granted events to stream created/deleted
2020-08-25 10:39:32 +03:00
Dimitrie Stefanescu 2ab449fce5 Merge branch 'izzy/gql-subscriptions' of https://github.com/Speckle-Next/Server into izzy/gql-subscriptions 2020-08-25 10:19:49 +03:00
izzy lyseggen 4fd1a6b480 test(subs): try supertest for the requests a la #3
`sendRequest` now uses the supertest method for executing and verifying requests

still using chai for the assertions with subscribe data and eventNum
as I wasn't sure what assertion lib we want to use if not chai
since supertest is just for api (if i'm understanding correctly??)
2020-08-24 17:47:03 +01:00
Dimitrie Stefanescu 02b660d8ae test(subs): keeps spawned process error logging in tests
so we can read them in the ci logs easier
2020-08-23 20:24:29 +03:00
izzy lyseggen 0723a69895 fix(streams): oops forgot some 2020-08-21 16:51:58 +01:00
izzy lyseggen b8c0bb4125 refactor(mutations): make inputs consistent
all mutations that require more than 1 input now has an input type 🌟
fixes #10
2020-08-21 16:45:21 +01:00
izzy lyseggen 0ef85374b3 style: clean redundant refs and formatting 2020-08-21 16:00:13 +01:00
izzy lyseggen f0db46bbca test(subs): more auth tests for subs
confirm you can't receive updates if you're not authorised for stream
2020-08-21 15:36:49 +01:00
izzy lyseggen d204902fcb style(subs-tests): formatting 2020-08-21 14:24:51 +01:00
izzy lyseggen be8e8be854 feat(subs): auth within subscripition resolvers 2020-08-21 14:11:10 +01:00
izzy lyseggen 397fb04689 feat(subs): add scope auth to branch & commit subs 2020-08-21 13:46:28 +01:00
izzy lyseggen 0674083f6c feat(auth-dirs): only allow on field definitons 2020-08-21 13:45:44 +01:00
izzy lyseggen 6db1bdcf42 feat(subs): simplify stream create/delete subs
as per discussion, remove `ownerId` input and only allow subscribing to
your own stream creation / deletion
2020-08-21 13:44:19 +01:00
izzy lyseggen 70c6b76b85 test(subs): stream sub auth tests
prevent sub for different user or with wrong scope
2020-08-21 10:12:44 +01:00
izzy lyseggen 773f86dcdc test(ci): make the child process windoze friendly 2020-08-21 09:54:18 +01:00
izzy lyseggen 1ded3a2a76 test(ci): merge in branch 'dim/sub-circle' 2020-08-21 09:52:47 +01:00
izzy lyseggen 75cddd1260 test(subs): permission granted and revoked tests 2020-08-21 09:38:48 +01:00
Dimitrie Stefanescu a9ad5cbd0b test(ci): peppered await sleep(500) before every mutation 2020-08-21 11:24:37 +03:00
Dimitrie Stefanescu 8fd0f7e580 test(ci): trying out new spawning mechanism for sub test 2020-08-21 11:16:02 +03:00
izzy lyseggen f71287105a feat(subs): permission subs resolvers
don't know where they vanished to, but they're fixed and back now
2020-08-21 07:44:57 +01:00
izzy lyseggen dc4d8de77e Merge branch 'izzy/gql-subscriptions' of https://github.com/Speckle-Next/Server into izzy/gql-subscriptions 2020-08-20 15:26:30 +01:00
izzy lyseggen 22248bf538 test(subs): basic tests for all subscriptions
did somebody say TESTS 🥳
2020-08-20 15:26:17 +01:00
Dimitrie Stefanescu d40258cfe6 test(circleci): added test server runner separately 2020-08-20 15:39:02 +03:00
izzy lyseggen ac58d4f53a Revert "test(subs): bump to timeout 7k"
Didn't work.

This reverts commit 41153f1aeb.
2020-08-20 10:42:48 +01:00
izzy lyseggen 41153f1aeb test(subs): bump to timeout 7k to pacify circle ci 2020-08-20 10:36:05 +01:00
izzy lyseggen 762f369c56 test(subs): bump timeout to 5k bc steve jobs's 👻 2020-08-20 10:08:43 +01:00
izzy lyseggen 99f473873a fix(subs): remove redundant stream sub defs 2020-08-20 10:00:26 +01:00
izzy lyseggen c6db01420a test(subs): merge branch 'dim/gql-subscriptions' 2020-08-20 09:50:15 +01:00
Dimitrie Stefanescu 7a03c884ea fix(tests/subscriptions): fixes port conflict in subscription tests when ran from all tests command 2020-08-20 11:31:25 +03:00
Dimitrie Stefanescu 29dfa57d82 test(subscriptions): subscription tests now run independently 🙌 2020-08-20 11:26:51 +03:00
Dimitrie Stefanescu e4f213d4f6 test(subscriptions): works with separate run test server, figuring out starting it from test file
doing `npm run dev:server:test` and running the test with that server over port 3000 works. seems
like ws upgrade requests are not honoured when the server is started from the test file (on 3001).
2020-08-20 10:27:48 +03:00
Dimitrie Stefanescu 45d73e3a47 test(subscriptions): first working scaffold of subscriptions test! 2020-08-19 21:47:48 +03:00
izzy lyseggen c406abe5ec feat(subs): subs for branch & commit mutations 🥳
also applied @hasRole and @hasScope directives to mutations in this schema
2020-08-19 15:46:40 +01:00
izzy lyseggen 6c3e8b138d feat(subs): rename stream subs & apply directives
- `streamCreated` -> `userStreamCreated
- `streamDeleted` -> `userStreamDeleted`

this is to better show that this is user bound (takes owner id)
rather than stream bound

- @hasRole and @hasScope have been reapplied to each field
2020-08-19 15:45:21 +01:00
izzy lyseggen 222f932547 feat(auth-dirs): revert @hasScope and @hasRole
make these field definition only again so it works for now.
the obj directives problem is more messy than i thought 🙃
2020-08-19 15:41:42 +01:00
izzy lyseggen eabfa99651 feat(auth-dirs): remove @isAuthorized directives
was decided that these were a bit too much.
auth will be reverted to `authorizeResolver` within the resolvers
2020-08-19 15:39:52 +01:00