Activity for Luke Cann

  • Luke Cann Luke Cann modified a comment on merge request #47

    Hi Jeff, I was eventually able to write a test for 427. I've rebased on master and squashed my changes down to three commits, one for each issue. This includes the changes to changes.xml that you requested. I looked through the dev guide. It seems to contain everything you've walked me through here at-least. It may be usefull to have some notes on code style. Most things, such as brackets and indentation is pretty clear at first glance but other things may not be.

  • Luke Cann Luke Cann modified a comment on merge request #47

    Hi Jeff, I was eventually able to write a test for 427. I've rebased on master and squashed my changes down to three commits, one for each issue. This includes the changes to changes.xml that you requested. I looked through the dev guide. It seems to contain everything you've walked me through here at-least. It may be usefull to have some notes on code style. Most things, such as brackets and indentation is pretty clear at first glance but other things may not be.

  • Luke Cann Luke Cann posted a comment on merge request #47

    Hi Jef, I was eventually able to write a test for 427. I've rebased on master and squashed my changes down to three commits, one for each issue. This includes the changes to changes.xml that you requested. I looked through the dev guide. It seems to contain everything you've walked me through here at-least. It may be usefull to have some notes on code style. Most things, such as brackets and indentation is pretty clear at first glance but other things may not be.

  • Luke Cann Luke Cann committed [c3745e]

    B427: Fix normalization of schemas in DatabaseDataSet

  • Luke Cann Luke Cann committed [593ee9]

    B426: Inherit case sensitivity in FilteredDataSet

  • Luke Cann Luke Cann committed [a241f8]

    B428: Fix value casting in IsActualWithinToleranceOfExpectedTimestampValueComparer

  • Luke Cann Luke Cann committed [a922b1]

    Fixed normalisation of schemas in regard to case sensitivity in DatabaseDataSet SchemaSet

  • Luke Cann Luke Cann committed [7b8254]

    B426: Inherit case sensitivity of wrapped set in FilteredDataSet

  • Luke Cann Luke Cann committed [3a2d50]

    B428: Cast values to expected type in IsActualWithinToleranceOfExpectedTimestampValueComparer

  • Luke Cann Luke Cann committed [12b8b1]

    B427: Add case sensitivity test for DatabaseDataSet

  • Luke Cann Luke Cann committed [0c374a]

    B427: Add case sensitivity test for DatabaseDataSet

  • Luke Cann Luke Cann committed [478a45]

    B426: Inherit case sensitivity of wrapped set in FilteredDataSet

  • Luke Cann Luke Cann committed [aa84b5]

    Fixed normalisation of schemas in regard to case sensitivity in DatabaseDataSet SchemaSet

  • Luke Cann Luke Cann committed [082790]

    B428: Cast values to expected type in IsActualWithinToleranceOfExpectedTimestampValueComparer

  • Luke Cann Luke Cann posted a comment on merge request #47

    Unfortunatly i am using Assertion.assertWithValueComparer directly instead of DefaultPrepAndExpectedTestCase so that commit doesn't fix the issue for me. I've added tests to 426 & 428 but i'm having a hard time with 427. I wanted to add the test to DatabaseDataSet_MultiSchemaTest but the issue doesn't seem to be reproducable with the H2 DB. I'll try again later with the mssql ITs as that is where i initially found the issue.

  • Luke Cann Luke Cann committed [01bf22]

    B428: Add test to verify converting of string values in IsActualWithinToleranceOfExpectedTimestampValueComparer

  • Luke Cann Luke Cann committed [b89a19]

    B428: Refactor casting of values in IsActualWithinToleranceOfExpectedTimestampValueComparer

  • Luke Cann Luke Cann committed [77f8ac]

    IsActualWithinToleranceOfExpectedTimestampValueComparer now casts to the expected type using the dataType provided to the comparer instead of assuming the values will be Timestamps.

  • Luke Cann Luke Cann committed [f3d4a5]

    B426: FilteredDataSet does not inherit case sensitivity of wrapped set

  • Luke Cann Luke Cann committed [f4fff9]

    Fixed FilteredDataSet not inheriting the case sensitivity of the data set it wrapped

  • Luke Cann Luke Cann committed [503ad4]

    Fixed normalisation of schemas in regard to case sensitivity in DatabaseDataSet SchemaSet

  • Luke Cann Luke Cann posted a comment on merge request #47

    Hi Jeff, I've raised bugs for all three of the fixes: 426: FilteredDataSet does not inherit case sensitivity of wrapped set 427: DatabaseDataSet SchemaSet is incorrectly normalizing schema names in regard to case sensitivity 428: IsActualWithinToleranceOfExpectedTimestampValueComparer assumes values are represented as Timestamps I'll implement tests for each one and commit with the relevant bug reference. I've done 426 and i'll get to the others soon. This issue came about when i was trying to use...

  • Luke Cann Luke Cann created ticket #428

    IsActualWithinToleranceOfExpectedTimestampValueComparer assumes values are represented as Timestamps

  • Luke Cann Luke Cann created ticket #427

    DatabaseDataSet SchemaSet is incorrectly normalizing schema names in regard to case sensitivity

  • Luke Cann Luke Cann committed [fd1a28]

    B426: FilteredDataSet does not inherit case sensitivity of wrapped set

  • Luke Cann Luke Cann created ticket #426

    FilteredDataSet does not inherit case sensitivity of wrapped set

  • Luke Cann Luke Cann committed [04146b]

    IsActualWithinToleranceOfExpectedTimestampValueComparer now casts to the expected type using the dataType provided to the comparer instead of assuming the values will be Timestamps.

  • Luke Cann Luke Cann created merge request #47

    Case sensitivity fixes in filtered data sets and database data set schema sets

  • Luke Cann Luke Cann committed [a76425]

    Fixed normalisation of schemas in regard to case sensitivity in DatabaseDataSet SchemaSet

  • Luke Cann Luke Cann committed [e16857]

    Fixed FilteredDataSet not inheriting the case sensitivity of the data set it wrapped

1
MongoDB Logo MongoDB