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.

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

[TT-1624] Make sure that each Solidity change has a corresponding Solidity Review issue in Jira #14417

Draft
wants to merge 33 commits into
base: develop
Choose a base branch
from

Conversation

Tofel
Copy link
Contributor

@Tofel Tofel commented Sep 13, 2024

Context: https://smartcontract-it.atlassian.net/browse/TT-1624

Tests executed on local:

  • no PR issue in changeset -> FAILURE
  • Solidity Review in changeset -> do not create new one -> do not update changset -> expect: PASS
  • No Solidity Review in changset -> no Open exists -> create new one -> update changeset -> expect: PASS
  • No Solidity Review in changset -> Open exists -> use existing -> update changeset -> expect: PASS
  • No Solidity Review in changset -> no Open exists -> cannot find template -> expect: FAILURE
  • No Solidity Review in changset -> no Open exists -> cannot create new issue -> expect: FAILURE
  • No Solidity Review in changset -> no Open exists -> cannot copy child issues -> close parent issue -> expect: FAILURE
  • No Solidity Review in changset -> no Open exists -> cannot copy 2nd template -> close 1st cloned issue -> close parent issue -> expect: FAILURE
  • No Solidity Review in changset -> no Open exists -> create new one -> cannot link epics to parent -> expect: FAILURE
  • No Solidity Review in changset -> no Open exists -> create new one -> cannot link to PR issue -> expect: FAILURE
  • No Solidity Review in changset -> more than one Open exists -> expect: FAILURE
  • Solidity Review in changset -> doesn't exist in JIRA -> expect: FAILURE

Tests executed in CI:

  • no PR issue in changeset -> FAILURE
  • Solidity Review in changeset -> do not create new one -> do not update changset -> expect: PASS
  • No Solidity Review in changset -> no Open exists -> create new one -> update changeset -> expect: PASS
  • No Solidity Review in changset -> Open exists -> use existing -> update changeset -> expect: PASS

Regression tests executed in CI:

  • jira issue can be extracted from PR title, etc
  • fix version is set correctly for core changes
  • artifact pipeline correctly processes changesets

Example of a run, where we added existing Solidity Review issue to the changeset: https://github.com/smartcontractkit/chainlink/actions/runs/10851165446/job/30114271721
Example of a run, where we add a newly created Solidity Review issue to the changeset: https://github.com/smartcontractkit/chainlink/actions/runs/10851584999/job/30115688207
Example of a run, where Solidity Review was already in the changeset: https://github.com/smartcontractkit/chainlink/actions/runs/10851705586/job/30116067584

Missing:

  • add tests for new functions

Copy link
Contributor

github-actions bot commented Sep 13, 2024

Static analysis results are available

Hey @Tofel, you can view Slither reports in the job summary here or download them as artifact here.

Please check them before merging and make sure you have addressed all issues.

@Tofel Tofel changed the title Tt 1624 solidity review jira each pr [TT-1624] solidity review jira each pr Sep 13, 2024
@Tofel Tofel force-pushed the tt_1624_Solidity_Review_jira_each_pr branch 2 times, most recently from c5a418d to f0f2e16 Compare September 13, 2024 14:28
@Tofel Tofel force-pushed the tt_1624_Solidity_Review_jira_each_pr branch from f0f2e16 to c42319d Compare September 13, 2024 14:30
@Tofel Tofel force-pushed the tt_1624_Solidity_Review_jira_each_pr branch from b0d7ce0 to 09e118b Compare September 13, 2024 14:40
@Tofel Tofel force-pushed the tt_1624_Solidity_Review_jira_each_pr branch from aa74cdf to 74c0601 Compare September 13, 2024 14:48
@Tofel Tofel changed the title [TT-1624] solidity review jira each pr [TT-1624] Make sure that each Solidity change has a corresponding Solidity Review issue in Jira Sep 13, 2024
@Tofel Tofel force-pushed the tt_1624_Solidity_Review_jira_each_pr branch 3 times, most recently from 56a1058 to a87044e Compare September 16, 2024 10:07
Copy link
Contributor

github-actions bot commented Sep 16, 2024

Solidity Review Jira issue

Hey @Tofel, we have linked this PR to a Jira issue for Solidity Review!

This is a new feature, that's currently in pilot phase, so please make sure that the linkage is correct. In a contrary case, please update it manually in JIRA and replace Solidity Review issue key in the changeset file with the correct one.
And please reach out to the Test Tooling team and notify them about any issues you encounter.

Any changes to the Solidity Review Jira issue should be reflected in the changeset file. If you need to update the issue key, please do so manually in the following changeset file: contracts/.changeset/wet-monkey-signs.md

This PR has been linked to Solidity Review Jira issue: TT-1707

@Tofel Tofel force-pushed the tt_1624_Solidity_Review_jira_each_pr branch from a87044e to 0e360b0 Compare September 16, 2024 10:10
@Tofel Tofel force-pushed the tt_1624_Solidity_Review_jira_each_pr branch from 0e360b0 to a525e1a Compare September 16, 2024 10:19
Copy link
Contributor

@scheibinger scheibinger left a comment

Choose a reason for hiding this comment

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

Posted a few inline comments and questions.

.github/scripts/jira/package.json Outdated Show resolved Hide resolved
@Tofel Tofel force-pushed the tt_1624_Solidity_Review_jira_each_pr branch from 10c2589 to d33f6e9 Compare September 19, 2024 17:11
@Tofel Tofel force-pushed the tt_1624_Solidity_Review_jira_each_pr branch from 46ddf26 to e262dd6 Compare September 19, 2024 17:16
@Tofel Tofel force-pushed the tt_1624_Solidity_Review_jira_each_pr branch from e262dd6 to 4116108 Compare September 19, 2024 17:18
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants