search

MycroftAI / mycroft-core

Mycroft Core, the Mycroft Artificial Intelligence platform.
https://mycroft.ai
Apache License 2.0
6.48k stars 1.27k forks source link

Silence file does not exist error when clearing VK files #3093

Closed krisgesling closed 2 years ago

krisgesling commented 2 years ago

Description

When clearing VK test files - if some files didn't exist (because they weren't created) the bare rm command will report an error that the files didn't exist. We don't care, only that they are removed if they did exist.

How to test

Run 

mycroft-start vktest -t hello-world
mycroft-start vktest clear

Should receive an error without this patch. Nicer output with the patch applied.

codecov-commenter commented 2 years ago

Codecov Report

Merging #3093 (22a50b0) into dev (2d15fca) will not change coverage. The diff coverage is n/a.

:exclamation: Current head 22a50b0 differs from pull request most recent head 5e72326. Consider uploading reports for the commit 5e72326 to get more accurate results

@@           Coverage Diff           @@
##              dev    #3093   +/-   ##
=======================================
  Coverage   53.58%   53.58%           
=======================================
  Files         121      121           
  Lines       11128    11128           
=======================================
  Hits         5963     5963           
  Misses       5165     5165

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 2d15fca...5e72326. Read the comment docs.

devops-mycroft commented 2 years ago

Voight Kampff Integration Test Succeeded (Results)