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

Refactoring database_fixture.cpp to remove test case names. #1907

Open
1 of 17 tasks
manikey123 opened this issue Aug 13, 2019 · 1 comment
Open
1 of 17 tasks

Refactoring database_fixture.cpp to remove test case names. #1907

manikey123 opened this issue Aug 13, 2019 · 1 comment
Milestone
7.1.0 - Feature R...

Comments

@manikey123
Copy link
Contributor

User Story
Create a new fixture class that is derived from database_fixture, add a hook function there, every individual test case that uses that fixture can define their own parameters for initializing the app instance.
Reference:
#1903 (comment)
Impacts
Describe which portion(s) of BitShares Core may be impacted by your request. Please tick at least one box.

  • API (the application programming interface)
  • Build (the build process or something prior to compiled code)
  • CLI (the command line wallet)
  • Deployment (the deployment process after building such as Docker, Travis, etc.)
  • DEX (the Decentralized EXchange, market engine, etc.)
  • P2P (the peer-to-peer network for transaction/block propagation)
  • Performance (system or user efficiency, etc.)
  • Protocol (the blockchain logic, consensus, validation, etc.)
  • Security (the security of system or user data, etc.)
  • UX (the User Experience)
  • Other (please add below)

Additional Context (optional)
Add any other context about your request here.

CORE TEAM TASK LIST

  • Evaluate / Prioritize Feature Request
  • Refine User Stories / Requirements
  • Define Test Cases
  • Design / Develop Solution
  • Perform QA/Testing
  • Update Documentation
@manikey123
Copy link
Contributor Author

@abitmore plz assign the issue to me

@manikey123 manikey123 mentioned this issue Aug 13, 2019
Merged
@abitmore abitmore changed the title Refractoring database.cpp to remove test case names. Refractoring database_fixture.cpp to remove test case names. Aug 13, 2019
@abitmore abitmore added this to the 4.1.0 - Feature Release milestone Aug 13, 2019
@pmconrad pmconrad changed the title Refractoring database_fixture.cpp to remove test case names. Refactoring database_fixture.cpp to remove test case names. Sep 24, 2019
@pmconrad pmconrad mentioned this issue Nov 6, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
7.1.0 - Feature Release
Development

No branches or pull requests
2 participants
@abitmore @manikey123