Bugfix rows_complete() dropping non-tested columns

[yjunechoe]

Summary

Similar to #588 - ensures that rows_complete() preserves all columns instead of just the ones tested.

Now:

create_agent(small_table) %>% 
  rows_complete(c(a, b, c)) %>% 
  interrogate() %>% 
  get_data_extracts(1)
#> 
#> ── Interrogation Started - there is a single validation step ───────────────────────────────────────────
#> ✔ Step 1: OK.
#> 
#> ── Interrogation Completed ─────────────────────────────────────────────────────────────────────────────
#> # A tibble: 2 × 8
#>   date_time           date           a b             c     d e     f    
#>   <dttm>              <date>     <int> <chr>     <dbl> <dbl> <lgl> <chr>
#> 1 2016-01-06 17:23:00 2016-01-06     2 5-jdo-903    NA 3892. FALSE mid  
#> 2 2016-01-30 11:23:00 2016-01-30     1 3-dka-303    NA 2230. TRUE  high

Previously:

create_agent(small_table) %>% 
  rows_complete(c(a, b, c)) %>% 
  interrogate() %>% 
  get_data_extracts(1)
#> 
#> ── Interrogation Started - there is a single validation step ───────────────────────────────────────────
#> ✔ Step 1: OK.
#> 
#> ── Interrogation Completed ─────────────────────────────────────────────────────────────────────────────
#> # A tibble: 2 × 3
#>       a b             c
#>   <int> <chr>     <dbl>
#> 1     2 5-jdo-903    NA
#> 2     1 3-dka-303    NA

Rider: fixes a critical typo from #588 where I got the T/F flipped in tbl_rows_distinct_2() (interrogate_distinct() method for mysql and bigquery)

Related GitHub Issues and PRs

• Ref: Bugfix rows_distinct() dropping non-tested columns #588

Checklist

• I understand and agree to the Code of Conduct.
• I have listed any major changes in the NEWS.
• I have added testthat unit tests to tests/testthat for any new functionality.

[rich-iannone]

LGTM!!

[rich-iannone]

@yjunechoe this is great, thanks! You can merge whenever you like!