Skip to content

Add testcase to simulate bitcoin schema in leveldb #14125

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
Closed
wants to merge 2 commits into from
Closed

Add testcase to simulate bitcoin schema in leveldb #14125

wants to merge 2 commits into from

Conversation

MapleLaker
Copy link
Contributor

No description provided.

@fanquake fanquake added the Tests label Aug 31, 2018
@MapleLaker
Update dbwrapper_tests.cpp
9fa2ec8 
Modify test case of bitcoin schema leveldb
@laanwj
Copy link
Member

laanwj commented Sep 1, 2018

concept ack

travis fail is due to linting failure:

This diff appears to have added new lines with trailing whitespace.
The following changes were suspected:
diff --git a/src/test/dbwrapper_tests.cpp b/src/test/dbwrapper_tests.cpp
@@ -44,0 +45,89 @@ BOOST_AUTO_TEST_CASE(dbwrapper)
+
+        BOOST_CHECK_EQUAL(res.ToString(), in_block.ToString());
+        BOOST_CHECK_EQUAL(res.ToString(), in_transaction.ToString());
+        BOOST_CHECK_EQUAL(res.ToString(), in_utxo.ToString());
+        BOOST_CHECK_EQUAL(lastblockfilenumber, res_uint_32);
+        BOOST_CHECK_EQUAL(isInReindexing, res_bool);
+        BOOST_CHECK_EQUAL(lastblock_hash, res);
+        BOOST_CHECK_EQUAL(res_bool, in_file_bool);
^---- failure generated from test/lint/lint-whitespace.sh

Empact
Empact reviewed Sep 1, 2018
View reviewed changes
Empact
Empact reviewed Sep 1, 2018
View reviewed changes
Empact
Empact reviewed Sep 1, 2018
View reviewed changes
ghost
ghost suggested changes Sep 7, 2018
View reviewed changes
@DrahtBot DrahtBot closed this May 7, 2019
@DrahtBot
Copy link
Contributor

DrahtBot commented May 7, 2019

The last travis run for this pull request was 248 days ago and is thus outdated. To trigger a fresh travis build, this pull request should be closed and re-opened.

@DrahtBot DrahtBot reopened this May 7, 2019
@fanquake fanquake closed this Aug 4, 2019
@adamjonas adamjonas mentioned this pull request Oct 21, 2019
Merged
maflcko pushed a commit that referenced this pull request Oct 23, 2019
Merge #17206: test: Add testcase to simulate bitcoin schema in leveldb
c5ac7af 
4896bac Add testcase to simulate bitcoin schema in leveldb (MapleLaker)

Pull request description:

  Resurrecting #14125 with updates based on comments of closed PR

ACKs for top commit:
  laanwj:
    ACK 4896bac
  dongcarl:
    ACK 4896bac

Tree-SHA512: 3290ea7e1e998901d5ee8921d1d76cec399cae30ac1911a45b86826afed47cee1acf92bd6438f1fa11ed785a3b17abdcb1c169bc0419945eda9fe4c089d0b6eb
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jun 27, 2021
@PastaPastaPasta
Merge bitcoin#17206: test: Add testcase to simulate bitcoin schema in…
16f656e 
… leveldb

4896bac Add testcase to simulate bitcoin schema in leveldb (MapleLaker)

Pull request description:

  Resurrecting bitcoin#14125 with updates based on comments of closed PR

ACKs for top commit:
  laanwj:
    ACK 4896bac
  dongcarl:
    ACK 4896bac

Tree-SHA512: 3290ea7e1e998901d5ee8921d1d76cec399cae30ac1911a45b86826afed47cee1acf92bd6438f1fa11ed785a3b17abdcb1c169bc0419945eda9fe4c089d0b6eb
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jun 28, 2021
@PastaPastaPasta
Merge bitcoin#17206: test: Add testcase to simulate bitcoin schema in…
72dff24 
… leveldb

4896bac Add testcase to simulate bitcoin schema in leveldb (MapleLaker)

Pull request description:

  Resurrecting bitcoin#14125 with updates based on comments of closed PR

ACKs for top commit:
  laanwj:
    ACK 4896bac
  dongcarl:
    ACK 4896bac

Tree-SHA512: 3290ea7e1e998901d5ee8921d1d76cec399cae30ac1911a45b86826afed47cee1acf92bd6438f1fa11ed785a3b17abdcb1c169bc0419945eda9fe4c089d0b6eb
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jun 29, 2021
@PastaPastaPasta
Merge bitcoin#17206: test: Add testcase to simulate bitcoin schema in…
9bd39a0 
… leveldb

4896bac Add testcase to simulate bitcoin schema in leveldb (MapleLaker)

Pull request description:

  Resurrecting bitcoin#14125 with updates based on comments of closed PR

ACKs for top commit:
  laanwj:
    ACK 4896bac
  dongcarl:
    ACK 4896bac

Tree-SHA512: 3290ea7e1e998901d5ee8921d1d76cec399cae30ac1911a45b86826afed47cee1acf92bd6438f1fa11ed785a3b17abdcb1c169bc0419945eda9fe4c089d0b6eb
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 1, 2021
@PastaPastaPasta
Merge bitcoin#17206: test: Add testcase to simulate bitcoin schema in…
4b0f649 
… leveldb

4896bac Add testcase to simulate bitcoin schema in leveldb (MapleLaker)

Pull request description:

  Resurrecting bitcoin#14125 with updates based on comments of closed PR

ACKs for top commit:
  laanwj:
    ACK 4896bac
  dongcarl:
    ACK 4896bac

Tree-SHA512: 3290ea7e1e998901d5ee8921d1d76cec399cae30ac1911a45b86826afed47cee1acf92bd6438f1fa11ed785a3b17abdcb1c169bc0419945eda9fe4c089d0b6eb
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 1, 2021
@PastaPastaPasta
Merge bitcoin#17206: test: Add testcase to simulate bitcoin schema in…
a938656 
… leveldb

4896bac Add testcase to simulate bitcoin schema in leveldb (MapleLaker)

Pull request description:

  Resurrecting bitcoin#14125 with updates based on comments of closed PR

ACKs for top commit:
  laanwj:
    ACK 4896bac
  dongcarl:
    ACK 4896bac

Tree-SHA512: 3290ea7e1e998901d5ee8921d1d76cec399cae30ac1911a45b86826afed47cee1acf92bd6438f1fa11ed785a3b17abdcb1c169bc0419945eda9fe4c089d0b6eb
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 12, 2021
@PastaPastaPasta
Merge bitcoin#17206: test: Add testcase to simulate bitcoin schema in…
6add296 
… leveldb

4896bac Add testcase to simulate bitcoin schema in leveldb (MapleLaker)

Pull request description:

  Resurrecting bitcoin#14125 with updates based on comments of closed PR

ACKs for top commit:
  laanwj:
    ACK 4896bac
  dongcarl:
    ACK 4896bac

Tree-SHA512: 3290ea7e1e998901d5ee8921d1d76cec399cae30ac1911a45b86826afed47cee1acf92bd6438f1fa11ed785a3b17abdcb1c169bc0419945eda9fe4c089d0b6eb
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 13, 2021
@PastaPastaPasta
Merge bitcoin#17206: test: Add testcase to simulate bitcoin schema in…
db42453 
… leveldb

4896bac Add testcase to simulate bitcoin schema in leveldb (MapleLaker)

Pull request description:

  Resurrecting bitcoin#14125 with updates based on comments of closed PR

ACKs for top commit:
  laanwj:
    ACK 4896bac
  dongcarl:
    ACK 4896bac

Tree-SHA512: 3290ea7e1e998901d5ee8921d1d76cec399cae30ac1911a45b86826afed47cee1acf92bd6438f1fa11ed785a3b17abdcb1c169bc0419945eda9fe4c089d0b6eb
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 13, 2021
@PastaPastaPasta
Merge bitcoin#17206: test: Add testcase to simulate bitcoin schema in…
432f329 
… leveldb

4896bac Add testcase to simulate bitcoin schema in leveldb (MapleLaker)

Pull request description:

  Resurrecting bitcoin#14125 with updates based on comments of closed PR

ACKs for top commit:
  laanwj:
    ACK 4896bac
  dongcarl:
    ACK 4896bac

Tree-SHA512: 3290ea7e1e998901d5ee8921d1d76cec399cae30ac1911a45b86826afed47cee1acf92bd6438f1fa11ed785a3b17abdcb1c169bc0419945eda9fe4c089d0b6eb
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Dec 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
2 more reviewers

@Empact Empact Empact left review comments

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
Tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@MapleLaker @laanwj @DrahtBot @Empact @fanquake