Adds "quote" parameter to "accepted_values" test

[clausherther]

Adds quote parameter to accepted_values test. Closes #1873.

Users can now test for acceptable integer values by opting not to quote these values.

...
         - name: day_of_week
            description: 
            tests:
              - not_null
              - accepted_values:
                  values: [1, 2, 3, 4, 5, 6, 7]
                  quote: false
...

Results in:

with all_values as (
    select distinct
        day_of_week as value_field
    from dw.dim_date
),
validation_errors as (
    select
        value_field
    from 
        all_values
    where
        value_field not in (
            1,
            2,
            3,
            4,
            5,
            6,
            7
            )
)

select count(*)
from validation_errors

While the default behavior is to quote, e.g.

...
         - name: day_of_week_name
            description: 
            tests:
              - not_null
              - accepted_values:
                  values: [Sunday, Monday, Tuesday, Wednesday, Thursday, Friday, Saturday]
...

with all_values as (
    select distinct
        day_of_week_name as value_field
    from dw.dim_date
),
validation_errors as (
    select
        value_field
    from 
        all_values
    where
        value_field not in (
            'Sunday',
            'Monday',
            'Tuesday',
            'Wednesday',
            'Thursday',
            'Friday',
            'Saturday'
            )
)

select count(*)
from validation_errors

[cla-bot]

Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.

In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, don't hesitate to ping @drewbanin.

CLA has not been signed by users: @clausherther

[drewbanin]

Thanks! Glad to get this test working better - it's been a long time coming :)

I just kicked off the tests here. CLA looks good too (@cla-bot check).

Looking forward to merging this!

[cla-bot]

The cla-bot has been summoned, and re-checked this pull request!

[clausherther]

@drewbanin anything I need to be doing re: that BigQuery integration test failure? Looks like they all failed, so this may be just a CI thing, but wanted to make sure.

[drewbanin]

Hmmm, that's really strange! I see this error in the Azure logs:

The job encountered an internal error during execution and was unable to complete successfully.

So, this looks like a BQ blip, I think we're ok here :)

[drewbanin]

This LGTM! Thanks for your contribution @clausherther :)