search

RedHatQE / polarize

GNU General Public License v3.0
2 stars 2 forks source link

Implement the prefix and suffix for title #26

Closed rarebreed closed 7 years ago

rarebreed commented 7 years ago

Actually get the prefix and suffix working, and write a script to edit all the existing XML description file </p> </div> </div> <div class="comment"> <div class="user"> <a rel="noreferrer nofollow" target="_blank" href="https://github.com/rarebreed"><img src="https://avatars.githubusercontent.com/u/6633241?v=4" />rarebreed</a> commented <strong> 7 years ago</strong> </div> <div class="markdown-body"> <p>The hooks were already in place in the xml-config.xml file, and in the Configurator class, but the code wasn't actually using them. I also had to handle the case where if the prefix and/or suffix are being used, then the testcaseImportHandler method needed to be able to look up the XML definition file given the now modified title. Previously, I could use just the title alone, since it was easy to break up the title into the important path sections (packagename/method_name.xml). But adding the prefix now violated the path location (PREFIXpackagename/method_nameSUFFIX.xml). So I had to strip out the PREFIX and SUFFIX from the title in the message reply received from the message bus.</p> </div> </div> <div class="comment"> <div class="user"> <a rel="noreferrer nofollow" target="_blank" href="https://github.com/rarebreed"><img src="https://avatars.githubusercontent.com/u/6633241?v=4" />rarebreed</a> commented <strong> 7 years ago</strong> </div> <div class="markdown-body"> <p>Implemented in c6fbfca</p> </div> </div> <div class="page-bar-simple"> </div> <div class="footer"> <ul class="body"> <li>© <script> document.write(new Date().getFullYear()) </script> Githubissues.</li> <li>Githubissues is a development platform for aggregating issues.</li> </ul> </div> <script src="https://cdn.jsdelivr.net/npm/jquery@3.5.1/dist/jquery.min.js"></script> <script src="/githubissues/assets/js.js"></script> <script src="/githubissues/assets/markdown.js"></script> <script src="https://cdn.jsdelivr.net/gh/highlightjs/cdn-release@11.4.0/build/highlight.min.js"></script> <script src="https://cdn.jsdelivr.net/gh/highlightjs/cdn-release@11.4.0/build/languages/go.min.js"></script> <script> hljs.highlightAll(); </script> </body> </html>