refactor(bigquery): Clean up sample test resources, fix comments

Author: whuffman36

google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/BigQueryImpl.java

@@ -441,9 +441,6 @@ public com.google.api.services.bigquery.model.Job call() throws IOException {
           && createException.getCause() != null
           && createException.getCause().getMessage() != null) {
 
-        /*GoogleJsonResponseException createExceptionCause =
-        (GoogleJsonResponseException) createException.getCause();*/
-
         Pattern pattern = Pattern.compile(".*Already.*Exists:.*Job.*", Pattern.CASE_INSENSITIVE);
         Matcher matcher = pattern.matcher(createException.getCause().getMessage());
 

google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/QueryJobConfiguration.java

@@ -187,7 +187,7 @@ private Builder(com.google.api.services.bigquery.model.JobConfiguration configur
       this();
       JobConfigurationQuery queryConfigurationPb = configurationPb.getQuery();
       this.query = queryConfigurationPb.getQuery();
-      // Allows to get undeclaredqueryparameters in jobstatistics2
+      // Allows to get undeclared query parameters in jobstatistics
       if (queryConfigurationPb.getQueryParameters() == null
           && queryConfigurationPb.getParameterMode() != null) {
         parameterMode = queryConfigurationPb.getParameterMode();

google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/spi/v2/HttpBigQueryRpc.java

@@ -208,10 +208,8 @@ public Dataset createSkipExceptionTranslation(Dataset dataset, Map<Option, ?> op
             .insert(dataset.getDatasetReference().getProjectId(), dataset)
             .setPrettyPrint(false)
             .setFields(Option.FIELDS.getString(options));
-    for (Map.Entry<Option, ?> entry : options.entrySet()) {
-      if (entry.getKey() == Option.ACCESS_POLICY_VERSION && entry.getValue() != null) {
-        bqCreateRequest.setAccessPolicyVersion((Integer) entry.getValue());
-      }
+    if (options.containsKey(Option.ACCESS_POLICY_VERSION)) {
+      bqCreateRequest.setAccessPolicyVersion((Integer) options.get(Option.ACCESS_POLICY_VERSION));
     }
     return bqCreateRequest.execute();
   }

google-cloud-bigquery/src/test/java/com/google/cloud/bigquery/it/ITBigQueryTest.java

@@ -1334,7 +1334,7 @@ public void testUpdateDatasetWithSelectedFields() {
   }
 
   @Test
-  public void testUpdateDatabaseWithAccessPolicyVersion() throws IOException {
+  public void testUpdateDatasetWithAccessPolicyVersion() throws IOException {
     String accessPolicyDataset = RemoteBigQueryHelper.generateDatasetName();
     ServiceAccountCredentials credentials =
         (ServiceAccountCredentials) GoogleCredentials.getApplicationDefault();
@@ -1827,7 +1827,7 @@ public void testCreateDatasetWithAccessPolicyVersion() throws IOException {
   }
 
   @Test(expected = BigQueryException.class)
-  public void testCreateDatabaseWithInvalidAccessPolicyVersion() throws IOException {
+  public void testCreateDatasetWithInvalidAccessPolicyVersion() throws IOException {
     String accessPolicyDataset = RemoteBigQueryHelper.generateDatasetName();
     ServiceAccountCredentials credentials =
         (ServiceAccountCredentials) GoogleCredentials.getApplicationDefault();
@@ -4040,7 +4040,7 @@ public void testBQResultSetPagination() throws SQLException {
     assertEquals(300000, cnt); // total 300000 rows should be read
   }
 
-  // @Test - Temporarily disabling till https://github.com/googleapis/gax-java/issues/1712 or
+  @Test //- Temporarily disabling till https://github.com/googleapis/gax-java/issues/1712 or
   // b/235591056 are resolved
   public void testReadAPIIterationAndOrder()
       throws SQLException { // use read API to read 300K records and check the order
@@ -4240,7 +4240,7 @@ public void testCreateDefaultConnection() throws BigQuerySQLException {
     assertTrue(connection.close());
   }
 
-  // @Test - Temporarily disabling till https://github.com/googleapis/gax-java/issues/1712 or
+  @Test //- Temporarily disabling till https://github.com/googleapis/gax-java/issues/1712 or
   // b/235591056 are resolved
   public void testReadAPIConnectionMultiClose()
       throws

samples/snippets/src/test/java/com/example/bigquery/GrantViewAccessIT.java

@@ -94,6 +94,7 @@ public void tearDown() {
     DeleteTable.deleteTable(viewDatasetName, viewName);
     DeleteTable.deleteTable(viewDatasetName, tableName);
     DeleteDataset.deleteDataset(PROJECT_ID, datasetName);
+    DeleteDataset.deleteDataset(PROJECT_ID, viewDatasetName);
     // restores print statements in the original method
     System.out.flush();
     System.setOut(originalPrintStream);