ceedling 1.2.0 → 1.2.2

data/plugins/sanitize/sanitize.rake

@@ -47,11 +47,12 @@ namespace SANITIZE_SYM do
     )
   end
 
-  desc "Run single sanitized test ([*] test or source file name, no path)."
+  desc "Run single sanitized test ([*] test or source file name, with optional path)."
   task :* do
     message = "Oops! '#{SANITIZE_ROOT_NAME}:*' isn't a real task. " +
-              "Use a real test or source file name (no path) in place of the wildcard.\n" +
-              "Example: `ceedling #{SANITIZE_ROOT_NAME}:foo.c`"
+              "Use a real test or source file name in place of the wildcard, " +
+              "adding as much of its path as needed to identify one file among any same-named tests.\n" +
+              "Example: `ceedling #{SANITIZE_ROOT_NAME}:foo.c` or `ceedling #{SANITIZE_ROOT_NAME}:unit/foo.c`"
     @ceedling[:loginator].log( message, Verbosity::ERRORS )
   end
 
@@ -92,9 +93,8 @@ end
 # Use a rule to handle dynamic per-file tasks: sanitize:foo → build and run foo with sanitizers
 rule(/^#{SANITIZE_TASK_ROOT}\S+$/ => [
   proc do |task_name|
-    test = task_name.strip().sub(/^#{SANITIZE_TASK_ROOT}/, '')
-    EXTENSION_SOURCE.each { |ext| test = test.chomp(ext) }
-    test = PROJECT_TEST_FILE_PREFIX + test unless test.start_with?(PROJECT_TEST_FILE_PREFIX)
+    # Any directory typed ahead of the name is kept. See TestTaskName.
+    test = TestTaskName.test_name( task_name, root: SANITIZE_TASK_ROOT )
     @ceedling[:file_finder].find_test_file_from_name(test)
   end
 ]) do |test|

data/plugins/valgrind/lib/valgrind.rb

@@ -152,7 +152,7 @@ class Valgrind < Plugin
 
     test_name = arg_hash[:test_name]
     path      = report_filepath( test_name )
-    return unless @file_wrapper.exist?( path )
+    return unless report_written?( path )
 
     cause = ValgrindReport.first_error( @file_wrapper.read( path ), xml: xml_report_enabled? )
     return if cause.nil?
@@ -230,7 +230,14 @@ class Valgrind < Plugin
     return xml_report_enabled? ? xml_filepath( test_name ) : log_filepath( test_name )
   end
 
-  # Assembles the full Valgrind argument list for one test.
+  # Valgrind wrote the report moments ago in another process, so a miss is retried before
+  # it is believed
+  def report_written?(path)
+    return @file_wrapper.exist_with_retry?( path )
+  end
+
+  # Assembles the full Valgrind argument list for one test, making the directories its
+  # reports go in. A nested test's reports sit in subdirectories, and Valgrind makes none.
   #
   # Suppression files lead so a user's own :arguments entries can still override
   # anything they set. `${1}` is the test executable substitution token and is always
@@ -240,15 +247,15 @@ class Valgrind < Plugin
 
     arguments = Array( config[:valgrind_suppressions] ).map { |file| "--suppressions=\"#{file}\"" }
 
-    arguments << "--log-file=\"#{log_filepath( test_name )}\""
+    arguments << "--log-file=\"#{@file_wrapper.mkdir_for( log_filepath( test_name ) )}\""
 
     if xml_report_enabled?
       arguments << '--xml=yes'
-      arguments << "--xml-file=\"#{xml_filepath( test_name )}\""
+      arguments << "--xml-file=\"#{@file_wrapper.mkdir_for( xml_filepath( test_name ) )}\""
     end
 
     arguments += Array( config[:valgrind_arguments] )
-    arguments << '${1}'
+    arguments << '"${1}"'
 
     return arguments
   end
@@ -260,11 +267,11 @@ class Valgrind < Plugin
   def count_errors(test_name)
     path = report_filepath( test_name )
 
-    unless @file_wrapper.exist?( path )
+    unless report_written?( path )
       # A silent skip here reads exactly like a clean result. It is not one. The
       # memory error status of this test is unknown.
       msg = "Valgrind wrote no report for #{test_name} at #{path}. Memory error status is unknown."
-      @loginator.log( msg, Verbosity::COMPLAIN, LogLabels::WARNING )
+      @loginator.log( msg, Verbosity::COMPLAIN )
       return nil
     end
 

data/plugins/valgrind/valgrind.rake

@@ -48,11 +48,12 @@ namespace VALGRIND_SYM do
     )
   end
 
-  desc "Run single Valgrind test ([*] test or source file name, no path)."
+  desc "Run single Valgrind test ([*] test or source file name, with optional path)."
   task :* do
     message = "Oops! '#{VALGRIND_ROOT_NAME}:*' isn't a real task. " +
-              "Use a real test or source file name (no path) in place of the wildcard.\n" +
-              "Example: `ceedling #{VALGRIND_ROOT_NAME}:foo.c`"
+              "Use a real test or source file name in place of the wildcard, " +
+              "adding as much of its path as needed to identify one file among any same-named tests.\n" +
+              "Example: `ceedling #{VALGRIND_ROOT_NAME}:foo.c` or `ceedling #{VALGRIND_ROOT_NAME}:unit/foo.c`"
     @ceedling[:loginator].log( message, Verbosity::ERRORS )
   end
 
@@ -93,9 +94,8 @@ end
 # Use a rule to handle dynamic per-file tasks: valgrind:foo → build and run foo under Valgrind
 rule(/^#{VALGRIND_TASK_ROOT}\S+$/ => [
   proc do |task_name|
-    test = task_name.strip().sub(/^#{VALGRIND_TASK_ROOT}/, '')
-    EXTENSION_SOURCE.each { |ext| test = test.chomp(ext) }
-    test = PROJECT_TEST_FILE_PREFIX + test unless test.start_with?(PROJECT_TEST_FILE_PREFIX)
+    # Any directory typed ahead of the name is kept. See TestTaskName.
+    test = TestTaskName.test_name( task_name, root: VALGRIND_TASK_ROOT )
     @ceedling[:file_finder].find_test_file_from_name(test)
   end
 ]) do |test|

data/site-local/404.html

@@ -17,7 +17,7 @@
       
       
       <link rel="icon" href="/assets/images/favicon.png">
-      <meta name="generator" content="mkdocs-1.6.1, mkdocs-material-9.7.7">
+      <meta name="generator" content="mkdocs-1.6.1, mkdocs-material-9.8.0">
     
     
       
@@ -25,10 +25,10 @@
       
     
     
-      <link rel="stylesheet" href="/assets/stylesheets/main.ec1eaa64.min.css">
+      <link rel="stylesheet" href="/assets/stylesheets/main.57a0d79d.min.css">
       
         
-        <link rel="stylesheet" href="/assets/stylesheets/palette.ab4e12ef.min.css">
+        <link rel="stylesheet" href="/assets/stylesheets/palette.fb2fffe2.min.css">
       
       
 
@@ -80,30 +80,16 @@
     </div>
     <div data-md-component="announce">
       
-        <aside class="md-banner">
-          <div class="md-banner__inner md-grid md-typeset">
-            
-            
-  
-
-          </div>
-          
-        </aside>
-      
     </div>
     
       <div data-md-color-scheme="default" data-md-component="outdated" hidden>
         
           <aside class="md-banner md-banner--warning">
             <div class="md-banner__inner md-grid md-typeset">
-              
-  
-    You are not viewing the latest version.
+              You are not viewing the latest version.
     <a href="..//">
       <strong>Click here to go to latest.</strong>
     </a>
-  
-
             </div>
             <script>var el=document.querySelector("[data-md-component=outdated]"),base=new URL("/"),outdated=__md_get("__outdated",sessionStorage,base);!0===outdated&&el&&(el.hidden=!1)</script>
           </aside>
@@ -4256,10 +4242,10 @@
     
       
       
-      <script id="__config" type="application/json">{"annotate": null, "base": "/", "features": ["navigation.tabs", "navigation.sections", "navigation.top", "navigation.indexes", "navigation.path", "search.suggest", "search.highlight", "content.code.copy", "content.code.annotate"], "search": "/assets/javascripts/workers/search.2c215733.min.js", "tags": null, "translations": {"clipboard.copied": "Copied to clipboard", "clipboard.copy": "Copy to clipboard", "search.result.more.one": "1 more on this page", "search.result.more.other": "# more on this page", "search.result.none": "No matching documents", "search.result.one": "1 matching document", "search.result.other": "# matching documents", "search.result.placeholder": "Type to start searching", "search.result.term.missing": "Missing", "select.version": "Select version"}, "version": {"default": "latest", "provider": "mike"}}</script>
+      <script id="__config" type="application/json">{"annotate": null, "base": "/", "features": ["navigation.tabs", "navigation.sections", "navigation.top", "navigation.indexes", "navigation.path", "search.suggest", "search.highlight", "content.code.copy", "content.code.annotate"], "search": "/assets/javascripts/workers/search.a7633da7.min.js", "tags": null, "translations": {"clipboard.copied": "Copied to clipboard", "clipboard.copy": "Copy to clipboard", "search.result.more.one": "1 more on this page", "search.result.more.other": "# more on this page", "search.result.none": "No matching documents", "search.result.one": "1 matching document", "search.result.other": "# matching documents", "search.result.placeholder": "Type to start searching", "search.result.term.missing": "Missing", "select.version": "Select version"}, "version": {"default": "latest", "provider": "mike"}}</script>
     
     
-      <script src="/assets/javascripts/bundle.d7400e89.min.js"></script>
+      <script src="/assets/javascripts/bundle.c21346df.min.js"></script>
       
     
   </body>