Skip to content
This repository has been archived by the owner on Mar 11, 2022. It is now read-only.

Apply 2.x specific tests to CouchDB 3.x #608

Merged
merged 1 commit into from Apr 14, 2020

Conversation

jjrodrig
Copy link
Contributor

Checklist

  • Tick to sign-off your agreement to the Developer Certificate of Origin (DCO) 1.1
  • Added tests for code changes or test/build only changes
  • Updated the change log file (CHANGES.md|CHANGELOG.md) or test/build only changes
  • Completed the PR template below:

Description

During testing on new CouchDB 3.x, I've noticed that some tests are skipped. These tests only run in a CouchDB 2.x.

This PR extends the version checking to 3.x for those tests.

Approach

Schema & API Changes

No change

Security and Privacy

No change

Testing

The following tests should pass in Couchdb 2.x and 3.x

changes_dbChangesMustSuccessfullyReturnWithSeqInterval
changes_selector
changes_selectorAndMoreThanLimitNumberOfDocs
pull_filterDocIdsFromAnimalDb_twoDocShouldBePulled
pull_filterSelectorMammalFromAnimalDbUsingParameterizedFilter_eightDocShouldBePulled
pull_filterSelectorBirdFromAnimalDb_twoDocShouldBePulled
pull_filterSelectorMammalFromAnimalDbUsingParameterizedFilter_eightDocShouldBePulled
pull_filterSelectorSmallFromAnimalDbUsingIntegerFilter_eightDocShouldBePulled
pull_filterSelectorSmallFromAnimalDbUsingNullFilter_eightDocShouldBePulled

Monitoring and Logging

No change

Copy link
Contributor

@bessbd bessbd left a comment

Choose a reason for hiding this comment

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

LGTM, +1

Thank you for this change, @jjrodrig !

@bessbd
Copy link
Contributor

bessbd commented Apr 14, 2020

I'm about to merge this.

@bessbd bessbd merged commit 96ae3b1 into cloudant:master Apr 14, 2020
@jjrodrig jjrodrig deleted the support-couchdb3-testing branch April 14, 2020 20:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants