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

Allow user to test with local SmartAPI spec using Docker container #143

Merged
merged 13 commits into from
Apr 2, 2021
Merged

Allow user to test with local SmartAPI spec using Docker container #143

merged 13 commits into from
Apr 2, 2021

Conversation

kevinxin90
Copy link
Contributor

No description provided.

@kevinxin90
build: ⬆️ upgrade smartapi-kg dependency
8a178f4
@kevinxin90
feat: ✨ update local predicates file and smartapi specs
425093e
@kevinxin90
refactor: ♻️ refactor existing association and predicates code to ada…
b64cbbe 
…pt to new smartapi-kg
@kevinxin90
feat: ✨ add cron job to periodically pull from /predicates endpoint
94461b1 
#141
@kevinxin90
test: ✅ update local test
d9df00c
@kevinxin90
refactor: ♻️ refactor to match new syntax in query handler
a20c08d
@kevinxin90
feat: ✨ add /test/query endpoint to support user specified smartapi spec
c75e44d
@kevinxin90
feat: ✨ have bte only use a specified number of APIs
7fa1732
@kevinxin90
refactor: ♻️ add example smartapi file
30fbe86
@kevinxin90
build: ⬆️ upgrade smartapi-kg and trapi query handler
3e92240
@kevinxin90
docs: 📝 add documentation on how to test local SmartAPI spec using do…
d9700f3 
…cker image
@kevinxin90
feat: ✨ support restricting /predicates to only api names provide
9260ec9
@kevinxin90
chore(release): 2.1.0
5c1dc8a
@coveralls
Copy link

Pull Request Test Coverage Report for Build 712399313

  • 29 of 38 (76.32%) changed or added relevant lines in 6 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-2.2%) to 93.716%
Changes Missing Coverage Covered Lines Changed/Added Lines %
src/routes/v1/query_test.js 6 15 40.0%
Totals Coverage Status
Change from base Build 698942508: -2.2%
Covered Lines: 281
Relevant Lines: 295
💛 - Coveralls

@kevinxin90 kevinxin90 merged commit 0fdcd38 into master Apr 2, 2021
@tokebe tokebe deleted the fix_issue_115 branch October 25, 2023 20:03
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.
2 participants
@kevinxin90 @coveralls