Revert "cli: have gherkin emit uris relative to the cwd (#1672)"

This reverts commit 5a21c22.

Author: davidjgoss

CHANGELOG.md

@@ -24,9 +24,6 @@ Please see [CONTRIBUTING.md](https://github.com/cucumber/cucumber/blob/master/CO
 
 ### Fixed
 
-* All messages now emitted with project-relative `uri`s
-  ([#1534](https://github.com/cucumber/cucumber-js/issues/1534)
-  [#1672](https://github.com/cucumber/cucumber-js/pull/1672))
 * Json formatter now works with tagged examples
   ([#1621](https://github.com/cucumber/cucumber-js/issues/1621)
    [#1651](https://github.com/cucumber/cucumber-js/pull/1651))

src/cli/helpers.ts

@@ -2,6 +2,7 @@ import _ from 'lodash'
 import ArgvParser from './argv_parser'
 import ProfileLoader from './profile_loader'
 import shuffle from 'knuth-shuffle-seeded'
+import path from 'path'
 import { EventEmitter } from 'events'
 import PickleFilter from '../pickle_filter'
 import { EventDataCollector } from '../formatter/helpers'
@@ -68,7 +69,10 @@ export async function parseGherkinMessageStream({
       if (doesHaveValue(envelope.parseError)) {
         reject(
           new Error(
-            `Parse error in '${envelope.parseError.source.uri}': ${envelope.parseError.message}`
+            `Parse error in '${path.relative(
+              cwd,
+              envelope.parseError.source.uri
+            )}': ${envelope.parseError.message}`
           )
         )
       }

src/cli/index.ts

@@ -205,7 +205,6 @@ export default class Cli {
       {
         defaultDialect: configuration.featureDefaultLanguage,
         newId,
-        relativeTo: this.cwd,
       }
     )
     const pickleIds = await parseGherkinMessageStream({

src/formatter/helpers/issue_helpers_spec.ts

@@ -11,7 +11,7 @@ async function testFormatIssue(sourceData: string): Promise<string> {
   const sources = [
     {
       data: sourceData,
-      uri: 'a.feature',
+      uri: 'project/a.feature',
     },
   ]
   const supportCodeLibrary = getBaseSupportCodeLibrary()

src/formatter/helpers/test_case_attempt_parser.ts

@@ -5,6 +5,7 @@ import {
   getGherkinStepMap,
 } from './gherkin_document_parser'
 import { getPickleStepMap, getStepKeyword } from './pickle_parser'
+import path from 'path'
 import * as messages from '@cucumber/messages'
 import { ITestCaseAttempt } from './event_data_collector'
 import StepDefinitionSnippetBuilder from '../step_definition_snippet_builder'
@@ -137,7 +138,7 @@ export function parseTestCaseAttempt({
     gherkinDocument
   )
   const pickleStepMap = getPickleStepMap(pickle)
-  const relativePickleUri = pickle.uri
+  const relativePickleUri = path.relative(cwd, pickle.uri)
   const parsedTestCase: IParsedTestCase = {
     attempt: testCaseAttempt.attempt,
     name: pickle.name,

src/formatter/helpers/usage_helpers/index.ts

@@ -1,5 +1,6 @@
 import _ from 'lodash'
 import { getPickleStepMap } from '../pickle_parser'
+import path from 'path'
 import { getGherkinStepMap } from '../gherkin_document_parser'
 import * as messages from '@cucumber/messages'
 import StepDefinition from '../../../models/step_definition'
@@ -72,7 +73,7 @@ function buildMapping({
         const match: IUsageMatch = {
           line: gherkinStep.location.line,
           text: pickleStep.text,
-          uri: testCaseAttempt.pickle.uri,
+          uri: path.relative(cwd, testCaseAttempt.pickle.uri),
         }
         const { duration, status } = testCaseAttempt.stepResults[testStep.id]
         if (!unexecutedStatuses.includes(status) && doesHaveValue(duration)) {

src/formatter/json_formatter.ts

@@ -1,6 +1,7 @@
 import _ from 'lodash'
 import Formatter, { IFormatterOptions } from './'
 import { formatLocation, GherkinDocumentParser, PickleParser } from './helpers'
+import path from 'path'
 import * as messages from '@cucumber/messages'
 import {
   getGherkinExampleRuleMap,
@@ -135,7 +136,7 @@ export default class JsonFormatter extends Formatter {
       this.eventDataCollector.getTestCaseAttempts(),
       (testCaseAttempt: ITestCaseAttempt) => {
         if (!testCaseAttempt.worstTestStepResult.willBeRetried) {
-          const uri = testCaseAttempt.pickle.uri
+          const uri = path.relative(this.cwd, testCaseAttempt.pickle.uri)
           if (doesNotHaveValue(groupedTestCaseAttempts[uri])) {
             groupedTestCaseAttempts[uri] = []
           }

src/formatter/rerun_formatter.ts

@@ -1,5 +1,6 @@
 import _ from 'lodash'
 import Formatter, { IFormatterOptions } from './'
+import path from 'path'
 import { getGherkinScenarioLocationMap } from './helpers/gherkin_document_parser'
 import {
   doesHaveValue,
@@ -36,7 +37,7 @@ export default class RerunFormatter extends Formatter {
         if (
           worstTestStepResult.status !== messages.TestStepResultStatus.PASSED
         ) {
-          const relativeUri = pickle.uri
+          const relativeUri = path.relative(this.cwd, pickle.uri)
           const line = getGherkinScenarioLocationMap(gherkinDocument)[
             _.last(pickle.astNodeIds)
           ].line

src/pickle_filter.ts

@@ -1,4 +1,5 @@
 import _ from 'lodash'
+import path from 'path'
 import parse from '@cucumber/tag-expressions'
 import { getGherkinScenarioLocationMap } from './formatter/helpers/gherkin_document_parser'
 import { doesHaveValue, doesNotHaveValue } from './value_checker'
@@ -72,7 +73,7 @@ export class PickleLineFilter {
     featurePaths.forEach((featurePath) => {
       const match = FEATURE_LINENUM_REGEXP.exec(featurePath)
       if (doesHaveValue(match)) {
-        const uri = match[1]
+        const uri = path.resolve(cwd, match[1])
         const linesExpression = match[2]
         if (doesHaveValue(linesExpression)) {
           if (doesNotHaveValue(mapping[uri])) {

src/pickle_filter_spec.ts

@@ -91,7 +91,7 @@ describe('PickleFilter', () => {
             gherkinDocument,
           } = await parse({
             data: ['Feature: a', '', 'Scenario: b', 'Given a step'].join('\n'),
-            uri: 'features/b.feature',
+            uri: path.resolve(cwd, 'features/b.feature'),
           })
 
           // Act