Github user jorgebay commented on a diff in the pull request:

    https://github.com/apache/tinkerpop/pull/747#discussion_r150502204
  
    --- Diff: 
gremlin-test/src/test/java/org/apache/tinkerpop/gremlin/process/FeatureCoverageTest.java
 ---
    @@ -0,0 +1,131 @@
    +/*
    + * Licensed to the Apache Software Foundation (ASF) under one
    + * or more contributor license agreements.  See the NOTICE file
    + * distributed with this work for additional information
    + * regarding copyright ownership.  The ASF licenses this file
    + * to you under the Apache License, Version 2.0 (the
    + * "License"); you may not use this file except in compliance
    + * with the License.  You may obtain a copy of the License at
    + *
    + * http://www.apache.org/licenses/LICENSE-2.0
    + *
    + * Unless required by applicable law or agreed to in writing,
    + * software distributed under the License is distributed on an
    + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
    + * KIND, either express or implied.  See the License for the
    + * specific language governing permissions and limitations
    + * under the License.
    + */
    +package org.apache.tinkerpop.gremlin.process;
    +
    +import 
org.apache.tinkerpop.gremlin.process.traversal.step.branch.BranchTest;
    +import 
org.apache.tinkerpop.gremlin.process.traversal.step.branch.ChooseTest;
    +import 
org.apache.tinkerpop.gremlin.process.traversal.step.branch.OptionalTest;
    +import org.apache.tinkerpop.gremlin.process.traversal.step.filter.CoinTest;
    +import org.apache.tinkerpop.gremlin.process.traversal.step.filter.DropTest;
    +import 
org.apache.tinkerpop.gremlin.process.traversal.step.filter.FilterTest;
    +import org.apache.tinkerpop.gremlin.process.traversal.step.filter.IsTest;
    +import org.apache.tinkerpop.gremlin.process.traversal.step.filter.OrTest;
    +import org.apache.tinkerpop.gremlin.process.traversal.step.map.CountTest;
    +import org.apache.tinkerpop.gremlin.process.traversal.step.map.PathTest;
    +import org.apache.tinkerpop.gremlin.process.traversal.step.map.ProjectTest;
    +import 
org.apache.tinkerpop.gremlin.process.traversal.step.map.ValueMapTest;
    +import org.apache.tinkerpop.gremlin.process.traversal.step.map.VertexTest;
    +import 
org.apache.tinkerpop.gremlin.process.traversal.step.sideEffect.AggregateTest;
    +import 
org.apache.tinkerpop.gremlin.process.traversal.step.sideEffect.GroupCountTest;
    +import 
org.apache.tinkerpop.gremlin.process.traversal.step.sideEffect.InjectTest;
    +import 
org.apache.tinkerpop.gremlin.process.traversal.step.sideEffect.StoreTest;
    +import org.junit.Ignore;
    +import org.junit.Test;
    +
    +import java.io.BufferedReader;
    +import java.io.File;
    +import java.io.FileInputStream;
    +import java.io.InputStream;
    +import java.io.InputStreamReader;
    +import java.lang.reflect.Field;
    +import java.lang.reflect.Method;
    +import java.util.Arrays;
    +import java.util.HashSet;
    +import java.util.List;
    +import java.util.Set;
    +import java.util.regex.Matcher;
    +import java.util.regex.Pattern;
    +import java.util.stream.Collectors;
    +import java.util.stream.Stream;
    +
    +import static org.hamcrest.core.Is.is;
    +import static org.junit.Assert.assertEquals;
    +import static org.junit.Assert.assertThat;
    +
    +/**
    + * @author Stephen Mallette (http://stephen.genoprime.com)
    + */
    +public class FeatureCoverageTest {
    +
    +    private static Pattern scenarioName = 
Pattern.compile("^\\s*Scenario:\\s*(.*)$");
    +
    +    @Test
    +    @Ignore("As it stands we won't have all of these tests migrated 
initially so there is no point to running this in full - it can be flipped on 
later")
    +    public void shouldImplementAllProcessTestsAsFeatures() throws 
Exception {
    --- End diff --
    
    This is a good way to verify how much of the tests have been migrated, but 
as it should be run once / few times, maybe a script / gist outside of the repo 
would be better, to avoid maintaining this source code in the future.


---

Reply via email to