test: refactor test-fs-watch-stop-sync

* format test per project guide
* use listener that emits clear message
* use common.mustNotCall() to confirm different listener is not invoked
* add comment describing the test

PR-URL: #13689
Reviewed-By: David Cai <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>

Author: Trott

test/parallel/test-fs-watch-stop-sync.js

@@ -1,10 +1,21 @@
 'use strict';
-
 const common = require('../common');
-const assert = require('assert');
+
+// This test checks that the `stop` event is emitted asynchronously.
+//
+// If it isn't asynchronous, then the listener will be called during the
+// execution of `watch.stop()`. That would be a bug.
+//
+// If it is asynchronous, then the listener will be removed before the event is
+// emitted.
+
 const fs = require('fs');
 
-const watch = fs.watchFile(__filename, common.noop);
-watch.once('stop', assert.fail);  // Should not trigger.
+const listener = common.mustNotCall(
+  'listener should have been removed before the event was emitted'
+);
+
+const watch = fs.watchFile(__filename, common.mustNotCall());
+watch.once('stop', listener);
 watch.stop();
-watch.removeListener('stop', assert.fail);
+watch.removeListener('stop', listener);