Feature/improved graph selection

[drewbanin]

This works:

# - run tests for models in ./models/some_dir/**
# - run tests for all snowplow models EXCEPT snowplow_id_map and its children
$ dbt test --models test_project.some.dir.* snowplow.* --exclude 'snowplow_id_map+'

Models

• select models by filesystem hierarchy
• select models by package
• select parents of model
• select children of model
• exclude models by above criteria

Schema Testing

• source tests from inside of packages
• select tests by package
• select tests by model hierarchy
• select tests by parents of model
• select tests by children of model
• exclude tests by above criteria

Custom Data Testing

• source tests from inside of packages
• select tests by package
• select tests by model hierarchy
• select tests by parents of model
• select tests by children of model
• exclude tests by above criteria

Remaining Todo

• Remove this package selector
• allow --models to be take a space-separated string

[drewbanin]

This branch also fixes #266

[cmcarthur]

@drewbanin did you ever figure out why the tests were mysteriously passing when the code was actually broken?

This looks great. If we understand why the tests were incorrectly passing before, then

(Also, don't forget to add a line to the CHANGELOG)

[cmcarthur]

nitpicky but why not call this graph_file_name?

[drewbanin]

👍

[cmcarthur]

HOLY SHIT THESE ARE GREAT

[drewbanin]

@cmcarthur the typo in the update.sql file here never failed for a couple of reasons:

• 003_* tests run after the 001_* tests, so the table we were inserting into always existed
• The 003 test was set up as "run, update, run again". Since the update was a nop, the assertions after the "update" just re-asserted what we had confirmed before the update.
• We also failed to update the summary_expected table to be consistent with the newly inserted data.

Now in the "update", we 1) insert into the correct table and 2) change the value of summary_expected to be consistent w/ the newly inserted data here.

[drewbanin]

@cmcarthur great, going to add those changelog lines then merge later today

[drewbanin]

cc @jthandy