Thanks to visit codestin.com
Credit goes to github.com

Skip to content

merge: staging to main #23

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
Merged
jinyoungbang merged 23 commits into from
Jan 23, 2024
Merged

merge: staging to main #23

jinyoungbang merged 23 commits into from
Jan 23, 2024

Conversation

@jinyoungbang
Copy link
Contributor

@jinyoungbang jinyoungbang commented Jan 8, 2024 •
edited
Loading

What does this PR do?

  • Pushes v0.2 to production

Type of change

  • Fix: Bug fix (non-breaking change which fixes an issue)
  • Refactor: Any code refactoring
  • Chore: technical debt, workflow improvements
  • Feature: New feature (non-breaking change which adds functionality)
  • Documentation: This change requires a documentation update

Tests Performed

Screenshots

Additional Comments

jinyoungbang and others added 23 commits December 26, 2023 19:44
@jinyoungbang
feat: implement unit testing infrastructure
a607a46
@jinyoungbang
feat: create coverage ci pipeline
d98a4da
@jinyoungbang
fix: fix yaml format
cc7bd8d
@jinyoungbang
fix: combine steps
dc11339
@jinyoungbang
fix: add default aws region
ad0487b
@jinyoungbang
fix: fix coverage report path
7d2e2e6
@jinyoungbang
feat: implement boilerplate for lsting service test
1026acd
@jinyoungbang
feat: add create listing unit test
3ebe5c8
feat: Created API for Google Spread Sheet
1dbce6d
@jinyoungbang
refactor: change how test receives service modules
36477ba
@jinyoungbang
chore: remove googlelib from v0.2
4250966
@jinyoungbang
fix: bug fix on json body req
10403e8
@jinyoungbang
feat: create Makefile for coverage
e9f57a8
@jinyoungbang
test: complete listing service test suite
a03e9d6
@jinyoungbang
feat: create test suite for applicants API
0dc7814
@jinyoungbang
chore: change mock variable names
92b6f45
@jinyoungbang
test: add listings API + error handler test suite
6f199c4
@jinyoungbang
chore: modify coverage threshold to 0.8
4d4dbaf
@jinyoungbang
feat: add dynamo db test suite (incomplete)
a303fe6
@jinyoungbang
feat: add decorators unit test suite
f0add72
@jinyoungbang
test: add utils unit test suite
e969c4a
@jinyoungbang
Merge pull request #17 from whyphi/feat/unit-test-infra
63942e1 
feat: Implement Unit Testing Infrastructure
@jinyoungbang
Merge pull request #21 from whyphi/dev/v0.2
b077117 
merge: `v0.2` to `staging`
@github-actions
Copy link

github-actions bot commented Jan 8, 2024

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
652 599 92% 80% 🟢

New Files

File Coverage Status
tests/init.py 100% 🟢
tests/api/init.py 100% 🟢
tests/api/test_applicants.py 100% 🟢
tests/api/test_listings.py 100% 🟢
tests/services/init.py 100% 🟢
tests/services/test_applicant_service.py 100% 🟢
tests/services/test_listing_service.py 100% 🟢
tests/test_app.py 100% 🟢
tests/test_decorators.py 100% 🟢
tests/test_dynamodb.py 100% 🟢
tests/test_utils.py 100% 🟢
TOTAL 100% 🟢

Modified Files

File Coverage Status
chalicelib/api/listings.py 100% 🟢
chalicelib/services/ListingService.py 100% 🟢
TOTAL 100% 🟢

updated for commit: b077117 by action🐍

@jinyoungbang jinyoungbang marked this pull request as ready for review January 23, 2024 01:09
@jinyoungbang jinyoungbang merged commit ab34c98 into main Jan 23, 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.

2 participants

@jinyoungbang