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

Moving Bigtable TestColumnRangeFilter within unit test file. #1335

Merged
merged 1 commit into from
Jan 7, 2016

Conversation

dhermes
Copy link
Contributor

@dhermes dhermes commented Dec 31, 2015

Doing this to match the order that the tested filters occur in row.py.

@tseaver Sorry this happened, I think it might've been caused by fly-by merges? I know you said the tests should come in the same order as the filters, which come in the same order as the fields in the proto spec.


To "verify" the order

$ cat gcloud/bigtable/row.py | grep -e '^class'
class Row(object):
class RowFilter(object):
class _BoolFilter(RowFilter):
class SinkFilter(_BoolFilter):
class PassAllFilter(_BoolFilter):
class BlockAllFilter(_BoolFilter):
class _RegexFilter(RowFilter):
class RowKeyRegexFilter(_RegexFilter):
class RowSampleFilter(RowFilter):
class FamilyNameRegexFilter(_RegexFilter):
class ColumnQualifierRegexFilter(_RegexFilter):
class TimestampRange(object):
class TimestampRangeFilter(RowFilter):
class ColumnRangeFilter(RowFilter):
class ValueRegexFilter(_RegexFilter):
class ValueRangeFilter(RowFilter):
class _CellCountFilter(RowFilter):
class CellsRowOffsetFilter(_CellCountFilter):
class CellsRowLimitFilter(_CellCountFilter):
class CellsColumnLimitFilter(_CellCountFilter):
class StripValueTransformerFilter(_BoolFilter):
class ApplyLabelFilter(RowFilter):
$
$
$ cat gcloud/bigtable/test_row.py | grep -e '^class'
class TestRow(unittest2.TestCase):
class Test_BoolFilter(unittest2.TestCase):
class TestSinkFilter(unittest2.TestCase):
class TestPassAllFilter(unittest2.TestCase):
class TestBlockAllFilter(unittest2.TestCase):
class Test_RegexFilter(unittest2.TestCase):
class TestRowKeyRegexFilter(unittest2.TestCase):
class TestRowSampleFilter(unittest2.TestCase):
class TestFamilyNameRegexFilter(unittest2.TestCase):
class TestColumnQualifierRegexFilter(unittest2.TestCase):
class TestTimestampRange(unittest2.TestCase):
class TestTimestampRangeFilter(unittest2.TestCase):
class TestColumnRangeFilter(unittest2.TestCase):
class TestValueRegexFilter(unittest2.TestCase):
class TestValueRangeFilter(unittest2.TestCase):
class Test_CellCountFilter(unittest2.TestCase):
class TestCellsRowOffsetFilter(unittest2.TestCase):
class TestCellsRowLimitFilter(unittest2.TestCase):
class TestCellsColumnLimitFilter(unittest2.TestCase):
class TestStripValueTransformerFilter(unittest2.TestCase):
class TestApplyLabelFilter(unittest2.TestCase):

Doing this to match the order that the tested filters occur
in row.py.
@dhermes dhermes added hygiene api: bigtable Issues related to the Bigtable API. labels Dec 31, 2015
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 31, 2015
@tseaver
Copy link
Contributor

tseaver commented Jan 7, 2016

LGTM

dhermes added a commit that referenced this pull request Jan 7, 2016
Moving Bigtable TestColumnRangeFilter within unit test file.
@dhermes dhermes merged commit f284923 into googleapis:master Jan 7, 2016
@dhermes dhermes deleted the re-order-TestColumnRangeFilter branch January 7, 2016 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: bigtable Issues related to the Bigtable API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants