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

Fixes and Enhancements #110

Merged
merged 37 commits into from
Jan 16, 2024
Merged

Fixes and Enhancements #110

merged 37 commits into from
Jan 16, 2024

Conversation

nadeem-cs
Copy link
Contributor

@nadeem-cs nadeem-cs commented Jan 11, 2024 •
edited
Loading

Changes of this PR includes:

  • fix: issue while updating entries with assets
  • test: adds Sanity tests changes
  • fix: critical dependency update

@abhinav-from-contentstack
fix: fixed issue while updating entries with assets
d98b470
@abhinav-from-contentstack
feat: added api test
b17a988
@abhinav-from-contentstack
Merge pull request #96 from contentstack/bug/CS-42703-updating-entry-…
39ea129 
…with-file

Bug/cs 42703 updating entry with file
@abhinav-from-contentstack
fix: updated changelog and package lock file
db6b879
@abhinav-from-contentstack
Merge pull request #98 from contentstack/bug/CS-43088-update-changelog
04f5d26 
fix: updated changelog and package lock file
@sunil-lakshman
Implemented Management token support (#99)
9ed7a91 
* Implemented Management token support

* Fixed PR Comments
@harshithad0703
test: added sanity tests for user and contenttype
701bc1e
@harshithad0703
added a script in package.json
508aa7a
@harshithad0703
Merge pull request #100 from contentstack/test/cs-43014-cms-sanity-re…
52c8786 
…adiness

basic sanity tests for user and contenttype
@harshithad0703
test: added entries, assets, organization and stack tests in sanity
211773e
@harshithad0703
changed the decription for it in testcases
ce0e53b
@harshithad0703
Merge pull request #101 from contentstack/test/cs-43131-basic-sanity-…
cfb7ea1 
…tests

assets, entries, organization, stack tests are added to sanity
@harshithad0703
test: added taxonomy and terms tests in sanity folder
d56c5eb
@harshithad0703
removed const stack as the api is read from env
67f2a62
@harshithad0703
Merge pull request #102 from contentstack/test/cs-43154-sanity-test-t…
3c4ba30 
…axonomy-terms

sanity test for taxonomy and terms
@harshithad0703
fix: changed the url in transferOwnership
2f32924
@harshithad0703
test: removed the skipped test case from organization test suit
2e38341
@harshithad0703
test: added branches and aliases test suits to sanity folder
258b3a9
@harshithad0703
test: commented out the test case instead or removing
ca58beb
@harshithad0703
test: updated url path in unit test case
941b9e9
@harshithad0703
Merge pull request #105 from contentstack/test/cs-43284-sanity-tests-…
f7128e7 
…branches-aliases

cs-43284 sanity tests for branches aliases
@harshithad0703
test: added 1 more test case to branch test suit
11c4d74
@harshithad0703
test: added create branch test after stack creation test
122b4db
@harshithad0703
Merge pull request #108 from contentstack/test/add-sanity-tests
ef3995d 
test: added create branch test after stack creation test
@nadeem-cs
fix: 📦 updates axios version
4bfd437
@nadeem-cs
chore: update package version in package.json
b73e856
@nadeem-cs
Merge branch 'master' into next
adfadb1
@nadeem-cs
chore: 🔨 updates prerelease hook script
45698d0
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 11, 2024 •
edited
Loading

Coverage report for commit: 5b13208
File: coverage/clover.xml

Cover ┌─────────────────────────┐ Freq.
   0% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  10% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  20% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  30% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  40% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  50% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  60% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  70% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  80% │ █████████░░░░░░░░░░░░░░ │ 15.9%
  90% │ ███████████████████████ │ 43.2%
 100% │ ██████████████████████░ │ 40.9%
      └─────────────────────────┘
 *Legend:* █ = Current Distribution
Summary - Lines: 90.19% | Methods: 97.61% | Branches: 73.27%
FilesLinesMethodsBranches
lib/app/authorization
   index.js91.67%100.00%72.09%
lib/app/hosting
   deployment.js85.29%100.00%65.79%
   index.js80.70%100.00%59.09%
lib/app
   index.js92.78%100.00%66.12%
lib/app/installation
   index.js84.62%100.00%59.55%
lib/app/request
   index.js91.43%100.00%65.85%
lib
   contentstack.js100.00%100.00%100.00%
   contentstackClient.js100.00%100.00%85.00%
   contentstackCollection.js100.00%100.00%100.00%
   entity.js85.00%100.00%62.75%
lib/core
   Util.js96.36%100.00%83.64%
   concurrency-queue.js94.19%100.00%85.71%
   contentstackError.js100.00%100.00%100.00%
   contentstackHTTPClient.js98.08%100.00%93.75%
lib/organization
   index.js83.33%100.00%60.67%
lib/organization/teams
   index.js92.11%100.00%68.42%
lib/organization/teams/stackRoleMappings
   index.js89.29%100.00%58.14%
lib/organization/teams/teamUsers
   index.js93.33%75.00%93.75%
lib/query
   index.js80.43%100.00%67.27%
lib/stack/asset/folders
   index.js100.00%100.00%100.00%
lib/stack/asset
   index.js85.29%84.21%64.94%
lib/stack/auditlog
   index.js95.83%100.00%75.76%
lib/stack/branch
   compare.js81.82%100.00%71.88%
   index.js97.44%100.00%80.56%
   mergeQueue.js100.00%100.00%73.68%
lib/stack/branchAlias
   index.js86.21%100.00%65.85%
lib/stack/contentType/entry
   index.js91.04%100.00%73.77%
lib/stack/contentType
   index.js83.02%85.71%70.18%
lib/stack/deliveryToken
   index.js100.00%100.00%94.44%
lib/stack/environment
   index.js100.00%100.00%100.00%
lib/stack/extension
   index.js95.92%100.00%87.76%
lib/stack/globalField
   index.js94.12%100.00%83.78%
lib/stack
   index.js82.80%85.00%71.01%
lib/stack/label
   index.js100.00%100.00%100.00%
lib/stack/locale
   index.js100.00%100.00%100.00%
lib/stack/managementToken
   index.js100.00%100.00%94.44%
lib/stack/release
   index.js91.11%100.00%72.34%
lib/stack/roles
   index.js100.00%100.00%100.00%
lib/stack/taxonomy
   index.js92.59%100.00%75.00%
lib/stack/taxonomy/terms
   index.js92.31%100.00%72.55%
lib/stack/webhook
   index.js89.66%100.00%72.88%
lib/stack/workflow
   index.js89.09%100.00%72.31%
lib/stack/workflow/publishRules
   index.js100.00%100.00%100.00%
lib/user
   index.js94.29%100.00%82.14%

🤖 comment via lucassabreu/comment-coverage-clover

@nadeem-cs nadeem-cs changed the title Critical dependencies update Fixes and Enhancements Jan 11, 2024
harshithad0703
harshithad0703 approved these changes Jan 16, 2024
View reviewed changes
Copy link
Contributor

@harshithad0703 harshithad0703 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved.

@nadeem-cs nadeem-cs marked this pull request as ready for review January 16, 2024 10:18
@nadeem-cs nadeem-cs requested a review from a team as a code owner January 16, 2024 10:18
@nadeem-cs nadeem-cs merged commit 8fb3ef7 into master Jan 16, 2024
8 checks passed
sunil-lakshman pushed a commit that referenced this pull request Jun 26, 2024
@nadeem-cs
Merge pull request #110 from contentstack/next
6c6f448 
Fixes and Enhancements
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@harshithad0703 harshithad0703 harshithad0703 approved these changes

@aravindbuilt aravindbuilt aravindbuilt approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@nadeem-cs @aravindbuilt @harshithad0703 @abhinav-from-contentstack @sunil-lakshman