htmltest icon indicating copy to clipboard operation
htmltest copied to clipboard

Incorrect total number of documents reported

Open MarkvanMents opened this issue 2 years ago • 0 comments

Describe the bug

The total number of documents scanned is incorrectly reported if

  • Only one file is selected using FilePath option
  • Directories are excluded using IgnoreDirs option

To Reproduce

Steps to reproduce the behaviour:

  1. Run with config and options FilePath: "aSingleFile"

  2. See error tested 6951 documents - should be 1 document

  3. Run with config and options

IgnoreDirs:
- "_print"
  1. See error tested 6951 documents - should be 3469 documents as the rest are in the _print directory - (the _print directory is still being ignored as errors there are not reported). (I think this latter bug is a result of #181 ?)

Expected behaviour

I would expect to see the number of documents which were actually tested.

Actual behaviour

Always lists the total number of documents in the documentStore using

// CountDocuments : Return number of documents in hT document store
func (hT *HTMLTest) CountDocuments() int {
	return len(hT.documentStore.Documents)
}

Versions

  • OS: Windows 10/Ubuntu 16.04.7 LTS
  • htmltest: 0.16.0

Additional context

Add any other context about the problem here.

MarkvanMents avatar Apr 29 '22 13:04 MarkvanMents