Skip to content

Feat/testing#5

Merged
NivGreenstein merged 73 commits intomasterfrom
feat/testing
May 12, 2024
Merged

Feat/testing#5
NivGreenstein merged 73 commits intomasterfrom
feat/testing

Conversation

@NivGreenstein
Copy link
Copy Markdown
Contributor

@NivGreenstein NivGreenstein commented May 12, 2024

Question Answer
Bug fix ✔/✖
New feature ✔/✖
Breaking change ✔/✖
Deprecations ✔/✖
Documentation ✔/✖
Tests added ✔/✖
Chore ✔/✖

Related issues: #XXX , #XXX ...
Closes #XXX ...

Further information:

NivGreenstein and others added 30 commits May 6, 2024 13:11
@NivGreenstein
delete: remove mapcolonies template tests
2dcd506
@NivGreenstein
fix: added await to getApp
be68188
@NivGreenstein
feat: exporting getTiles query params
6ffad54
@NivGreenstein
feat: created requestSender for tiles tests
86b5a54
@NivGreenstein
feat: created tiles integration tests
51b2785
@NivGreenstein
chore: GetItemsQueryParams interface created and exported
256443b
@NivGreenstein
feat: integration tests created for items
60459e6
@NivGreenstein
chroe: created GetRoutesQueryParams and exported interface
75e9c34
@NivGreenstein
feat: created requestSender for route testing
bbb3ad6
@NivGreenstein
feat: created route integration testing
fc9245d
@NivGreenstein
added empty response test
f0b63d5
changed tile
444acb5
fixed test case
d2f887b
updated default password
f2d92db
added config to tests enviorment
e4d12a1
created dev scripts with mock data
7271dbb
fix: changed parameters according to the new mock data
8e4320e
feat(package.json): added dev:scripts script
4fb40b5
fix(tsconfig): added resolveJsonModule: true
d609b89
fix(openapi3): changed oneOf to anyOf
be1a615
fix: updated password
197a910
fixed zone
6c4af89
feat: exported query params interfaces
8f71971
feat: added types and testTimeout increased to 15000
0ffc2c7
fix: updated openapi3 to return the right type
f22552b
added dataSource.ts for typeorm migration
40332d9
feat: added latLon integration testing
6605d29
feat: added typeorm migration scripts
ce1efde
@NivGreenstein
fix: fixed test cases for missing required properties and invalid ones
5ac81d3
@NivGreenstein
fix: updated returned schema on /lookup/latLonToTile
e87b598
Niv Greenstein added 29 commits May 9, 2024 16:38
fix: changed type to correct one
1730143
chore: changed test from having for-loop to jest's test.each loop so …
01ddcf0 
…if one fails the other won't
feat: created new FeatureCollection type
ef277ba
fix(item.ts): updated type to extend @types/geojson Feature
bc3d3b4
fix(route.ts): updated type to extend @types/geojson Feature
cb54e19
fix(tile.ts): updated type to extend @types/geojson Feature
16752f7
fix: updated formatResponse returned object to FeatureCollection
02047cd
fix(itemManager): updated returned type
3be5025
delete(latLonManager): removed unused import
20b7549
fix(routeManager): updated returned type
6388265
fix(tileManager): updated returned type
13daf1f
delete: removed boilerplate code
de2a512
feat: created mock data for formatResponse unit testing
83c6c29
feat: created formatResponse unit testing
102f551
fix: fixed if statement
aa9a405
fix(item): removed properties. now we accept every key as a property
63fc5f9
fix(route): removed properties. now we accept every key as a property
4adaf78
fix(tile): removed properties. now we accept every key as a property
4b4d853
fix(itemManager): added optional chain as property now could be null
75f1a8a
fix(latLonManager): added optional chain as property now could be null
7c417b0
fix(tileManager): added optional chain as property now could be null
0eb518a
fix(routeManager): added optional chain as property now could be null
3a44952
fix: fixed functions types
4765db5
fix(latLonController): fixed returned type
f94cb57
fix: added "type" key to returned tilesSchema
203cc97
feat: added test cases
51ced62
fix: changed test to properly check the assignment of arr[i] = 0 and …
c3168c2 
…it's failure
delete: removed console.log
34df5a3
WIP : commited only to be deleted later
915b970
@NivGreenstein NivGreenstein merged commit aa18c2c into master May 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@NivGreenstein