-
Notifications
You must be signed in to change notification settings - Fork 20
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.
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
Structure 2 #130
Merged
Merged
Structure 2 #130
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Implement validation * Implement serialization (#101) * Implement validation for nested types
* Implement clone to the new core * Make linter happy
Refactor internal implementation
* Webpack setup and begin to setup tests * Add electron to dev dependencies * Improve Webpack output during tests * Add browser tests to Travis * Remove unecessary dependencies * Update readme about browser and support * Separate travis lifecycles * Attempt to avoid hanging electron test * Update coveralls
* Move current code to packages * Share prettier config * Add coveralls to root package * Fix Coveralls script * Add coverage to root package
* Begin to implement assertions * Migrate boolean validation assertions to use jest-structure * Assert no expected for valid/invalid matchers * Assert attribute path is passed * Add package to automatically add assertions to jest * Begin to write tests * Add tests for validating error messages * Reorganize and begin to write toHaveInvalidAttributes * Implement toHaveInvalidAttributes and cover with tests * Rename auto.js to extend-expect.js * Refactor checking of path presence and usage hint * Refactor and remove hint from validity check of toHaveInvalidAttributes * Remove hint from validity check of toHaveInvalidAttribute * Refactor checking of attribute validity * Refactor usage error and remove hint for non-usage-related errors * Replace usage of assertValid with expect().toBeValid() * Replace validitationMatchers with jest-structure in array specs * Migrate whole structure spec to use jest-structure * Rename validity assertions to contain 'structure' in the name * Fix coverage script * Add tests for when using arrayContaining with jest-structure * Add readme
* Fallback to default when value is null and not nullable * Update changelog * Update docs about nullable attributes * Refactor name of dynamic default instance variable
* Allow to pass custom identifier for dynamic types * Improve documentation wording * Improve changelog wording * Apply suggestions from code review Co-Authored-By: Leonardo Negreiros de Oliveira <negreirosleo12@gmail.com> * Update docs/schema-concept/circular-references-and-dynamic-types.md Co-Authored-By: Leonardo Negreiros de Oliveira <negreirosleo12@gmail.com>
* Allow to use custom setters and getters to attributes * Add docs about custom getters and setters * Allow to inherit custom accessors * reduce function complexity * Add docs about custom accessors and inheritance
* Update dependencies and Node * Generate build * Remove Node 8 from Travis build * Make linter happy
* Invert symlinks * Bump alpha patch
* Gix Gitbook refs * New organization * Remove introduction duplicate link * Prepare to release new alpha
* Add testing to docs and prettify md files * Fix formatting
* Improve docs * Improve verbiage
negreirosleo
approved these changes
Mar 31, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.