Skip to content

Draft: Test full naming sequence by chaining `addJob` and `generateFilename` in prep of #54

Kwon-Young Choi requested to merge kchoi/kstars:feat54/step5 into master

Test full naming sequence by chaining addJob and generateFilename

Outside of addJob and generateFilename, had to manually:

  • sanitize targetName
  • construct the directory part of the path by concatenating
    • SequenceJob::localDirectory
    • SequenceJob::directoryPostfix

The test is done in TestPlaceholderPath::testFullNamingSequence. Comparison between the generated path and results had to be change to a regex match because of the varying path result:

  • the path can now include a date because of the TimeStampEnabled parameter. The regex used to match a date is \d{4}-\d{2}-\d{2}T\d{2}-\d{2}-\d{2}.
  • Options::fitsDir() from kstars/ekos/capture/placeholderpath.cpp:206 results my home directory. The regex used to match it is .*.
  • The whole regex is enclosed in ^myregx$ to match the total string.
  • . dots in the path are escaped: \.
Edited by Kwon-Young Choi

Merge request reports