From 4bbc19c421a5da736828e2217d85803bfe5eebfc Mon Sep 17 00:00:00 2001
From: Kris Jusiak <kris@jusiak.net>
Date: Tue, 20 Dec 2022 18:32:01 +0000
Subject: [PATCH] :technologist: Output path to feature file and source for
 steps

Problem:
  Working with BDD tests can be challenging in large repositories
  because it is difficult to find the implementation of steps. Patterns
  of text substitution and multi-line strings make simple greps for step
  definitions insufficient.

Solution:
  Rather than outputting either the step location or the feature file
  location, we will instead output the feature file location and
  optionally the step definition location if available (uses $Given,
  $When, $Then)
---
 include/GUnit/GSteps.h | 27 +++++++++++++++++----------
 1 file changed, 17 insertions(+), 10 deletions(-)

diff --git a/include/GUnit/GSteps.h b/include/GUnit/GSteps.h
index a0ebcdb..d70a98c 100644
--- a/include/GUnit/GSteps.h
+++ b/include/GUnit/GSteps.h
@@ -592,20 +592,27 @@ class Steps : public ::testing::EmptyTestEventListener {
               throw StepIsAmbiguous{"STEP \"" + expectedStep.second->name +
                                     "\" is ambiguous!"};
             }
-            // update the curret step
+            // update the current step
             current_step_ = i;
             const auto name = given_step.second.first.name;
-            const auto full_file = given_step.second.first.file.empty()
-                                       ? file_
-                                       : given_step.second.first.file;
-            const auto file =
-                full_file.substr(full_file.find_last_of("/\\") + 1);
-            const auto line = expectedStep.second->line;
+            const auto file = given_step.second.first.file;
 
             std::cout << "\033[0;96m"
-                      << "[ " << std::right << std::setw(8) << name << " ] "
-                      << std::left << std::setw(60) << expectedStep.second->name
-                      << "# " << file << ":" << line << "\033[m" << '\n';
+                      << "[ " << std::right << std::setw(8) << name << " ] " << std::left << std::setw(60) << expectedStep.second->name << "# ";
+
+            {
+              const auto file = file_.substr(file_.find_last_of("/\\") + 1);
+              const auto line = expected_step["locations"].back()["line"].template get<int>();
+              std::cout << file << ":" << line;
+            }
+
+            if (not file.empty()) {
+              const auto line = given_step.second.first.line;
+              std::cout << " " << file << ":" << line;
+            }
+
+            std::cout << "\033[m" << '\n';
+
             info_.step = expectedStep.second->name;
             given_step.second.second(expectedStep.second->name,
                                      detail::make_table(expected_step));