Skip to content

Commit 9b1ed8a

Browse files
committed
chore: Add coverage-ignore annotations for untestable code
Annotate structurally untestable code paths: - main(): thin wrapper calling os.Exit via log.Fatal - run() goroutine loop: blocks forever on <-done - CmdMatch error branches: CmdMatch never returns errors
1 parent b9c78c9 commit 9b1ed8a

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

cissh.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ func run() error {
2626
done := make(chan bool, 1)
2727

2828
// Iterate through the server ports and spawn a Goroutine for each
29-
for portNumber := *startingPortPtr; portNumber < numListeners; portNumber++ {
29+
for portNumber := *startingPortPtr; portNumber < numListeners; portNumber++ { // coverage-ignore // blocks on <-done
3030
go sshlistners.GenericListener(myFakeDevice, portNumber, handlers.GenericCiscoHandler, done)
3131
}
3232

@@ -35,7 +35,7 @@ func run() error {
3535
return nil
3636
}
3737

38-
func main() {
38+
func main() { // coverage-ignore
3939
if err := run(); err != nil {
4040
log.Fatal(err)
4141
}

ssh_server/handlers/ciscohandlers.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ func GenericCiscoHandler(myFakeDevice *fakedevices.FakeDevice) {
4545
userInput, myFakeDevice.ContextSearch,
4646
)
4747
if err != nil {
48-
log.Println(err)
48+
log.Println(err) // coverage-ignore // CmdMatch never returns errors
4949
break
5050
}
5151

@@ -93,7 +93,7 @@ func GenericCiscoHandler(myFakeDevice *fakedevices.FakeDevice) {
9393
// Run userInput through the command matcher to look at supportedCommands
9494
match, matchedCommand, multipleMatches, err := utils.CmdMatch(userInput, myFakeDevice.SupportedCommands)
9595
if err != nil {
96-
log.Println(err)
96+
log.Println(err) // coverage-ignore // CmdMatch never returns errors
9797
break
9898
}
9999

0 commit comments

Comments
 (0)