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

fix: implement new metadata handling for Bot #226

Merged
merged 7 commits into from
Dec 16, 2021
Merged

Conversation

scolladon
Copy link
Owner

@scolladon scolladon commented Dec 14, 2021

What does this pull request contains


  • Added for new features.
  • Changed for changes in existing functionality.
  • Deprecated for soon-to-be removed features.
  • Removed for now removed features.
  • Fixed for any bug fixes.
  • Security in case of vulnerabilities.

Explain your changes


Change the metadata definition for Bot and BotVersion
Create a dedicated handler for this new metadata kind
Create test class for it
Decomissionate old test class and metadata configuration

Does this close any currently open issues?


closes #219

  • Jest test to check the fix is applied are added.

Any particular element to being able to test locally


Follow the description of the related issue in order to create the right dataset.
use reproduction playground branch issue/219 with the following command:

$sfdx sgd:source:delta -d -f "HEAD~2"

Any other comments?


Where has this been tested?


Operating System: Darwin Kernel Version 19.6.0: Tue Oct 12 18:34:05 PDT 2021; root:xnu-6153.141.43~1/RELEASE_X86_64

yarn version: 1.22.17

node version: v14.18.1

git version: 2.34.1

sfdx version: sfdx-cli/7.130.1 darwin-x64 node-v14.18.1

sgd plugin version: 4.11.0

@scolladon
Copy link
Owner Author

Be careful, it should be merged after #220 !!

@codecov
Copy link

codecov bot commented Dec 14, 2021

Codecov Report

Merging #226 (c2e7206) into master (bcacdaa) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #226   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           20        20           
  Lines          494       494           
=========================================
  Hits           494       494           
Impacted Files Coverage Δ
src/service/standardHandler.js 100.00% <100.00%> (ø)
src/service/typeHandlerFactory.js 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bcacdaa...c2e7206. Read the comment docs.

mehdicherf
mehdicherf previously approved these changes Dec 15, 2021
@codeclimate
Copy link

codeclimate bot commented Dec 16, 2021

Code Climate has analyzed commit c2e7206 and detected 0 issues on this pull request.

View more on Code Climate.

@scolladon scolladon merged commit cd8267e into master Dec 16, 2021
@scolladon scolladon deleted the fix/bot-metadata branch December 16, 2021 20:18
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.

Not adding Bot or BotVersion
2 participants