Skip to content
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

Jump to bottom

Port the rest of the unit tests from OCaml to F# #3369

Merged
merged 30 commits into from
Jan 4, 2022
Merged

Port the rest of the unit tests from OCaml to F# #3369

merged 30 commits into from
Jan 4, 2022

Conversation

pbiggar
Copy link
Member

@pbiggar pbiggar commented Jan 4, 2022 •
edited
Loading

  • Add test that you can't access local files
  • Add test for DarkInternal::checkPermission
  • Port DvalRepr date test
  • Add sampleDval for errors within list/records
  • Make DarkInternal metadata match OCaml values
  • Port TraceInput tests
  • make Passwords redact in the same way as before
  • Delete unported files which have been ported
  • Add more password tests
  • Delete test code from more DB tables
  • Port canvas tests
  • Convert integration test json files to fluid format

@pbiggar
Add passwd test
6adcfcc
@pbiggar
Add cleanup comment
5b1ed5b
@pbiggar
Clarify comment
6bc13e9
@pbiggar
Test DarkInternal access
d6eaf4b
@pbiggar
Remove test - adequately tested elsewhere
43d055e
@pbiggar
Port date test
32de27b
@pbiggar
Add more test cases
5768b23
@pbiggar
Make LibDarkInternal match the old values
a3736b7
@pbiggar
Port two TraceInputs tests
c4be3a6
@pbiggar
Port test that function results are stored
42a1108
@pbiggar
This isn't urgent
96fdd7e
@pbiggar
Sort tests
cf87629
@pbiggar
Note tests remaining to be done
4fe8e0a
@pbiggar
Delete a bunch of unported files which have been ported
2be7282
@pbiggar
Fix comment
8ce4c58
@pbiggar
Remove ported code
e185154
@pbiggar
Fix tests
775756d
@pbiggar
Split password libraries into their own file, add more tests
c0c08d5
@pbiggar
Remove ported code
10d2f41
@pbiggar
Sort DBs being cleared, catch a few we didn't have before
0d973a9
@pbiggar
Start canvas tests
c3f4019
@pbiggar
Add more of the Canvas Tests
0bf9e3f
@pbiggar
Finish porting rest of Canvas tests
ff1a07c
@pbiggar
Convert tests from the old serialization format to fluid
0ba9fb0
@pbiggar
Read from fluid json
47358ab
@pbiggar
Allow reading deleted handlers
8750efe 
This makes the canvasClone test work
@pbiggar
Fix test
914c9ce
@pbiggar
Fix formatting
38ab629
@pbiggar
Fix test locally and document how
d9fef23
@pbiggar
Fix racy test
eba42a5
@pbiggar pbiggar merged commit 2ace17b into main Jan 4, 2022
@pbiggar pbiggar deleted the paul/rest-of-tests branch January 4, 2022 13:50
@pbiggar pbiggar restored the paul/rest-of-tests branch January 4, 2022 13:55
@pbiggar pbiggar deleted the paul/rest-of-tests branch January 5, 2022 13:38
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
@pbiggar