fix: sonar cloud smells (#155)

## PR Checklist

Please check if your PR fulfills the following requirements:

- [x] The commit message follows our guidelines: CONTRIBUTING.md#commit
- [ ] Tests for the changes have been added (for bug fixes / features)
- [ ] Docs have been added / updated (for bug fixes / features)

## PR Type

What kind of change does this PR introduce?

<!-- Please check the one that applies to this PR using "x". -->

```
[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:
```

## What is the current behavior?

The Sonar cloud on the main branch reports six smells on the code.

The smells are about

- replacing || operator with ??
- returning this as the type of fluent API functions

## What is the new behavior?

- Logical Or operator (||) is replaced for coalescence operator (??)
- We set this as the return type of fluent functions

## Does this PR introduce a breaking change?

```
[ ] Yes
[x] No
```

## Other information


https://sonarcloud.io/project/issues?resolved=false&id=ngworker_lumberjack

packages/internal/console-driver/test-util/src/lib/noop-console/noop-console.module.ts

@@ -10,7 +10,7 @@ import { NoopConsole } from './noop-console.service';
  deps: [[new Optional(), new SkipSelf(), NoopConsole]],
  provide: lumberjackConsoleToken,
  useFactory: (maybeExistingInstance: NoopConsole | null): NoopConsole =>
- maybeExistingInstance || new NoopConsole(),
+ maybeExistingInstance ?? new NoopConsole(),
  },
  ],
 })

packages/internal/console-driver/test-util/src/lib/spy-console/spy-console.module.ts

@@ -9,7 +9,7 @@ import { SpyConsole } from './spy-console.service';
  {
  deps: [[new Optional(), new SkipSelf(), SpyConsole]],
  provide: lumberjackConsoleToken,
- useFactory: (maybeExistingInstance: SpyConsole | null): SpyConsole => maybeExistingInstance || new SpyConsole(),
+ useFactory: (maybeExistingInstance: SpyConsole | null): SpyConsole => maybeExistingInstance ?? new SpyConsole(),
  },
  ],
 })

packages/internal/test-util/src/lib/object-driver/object.service.ts

@@ -5,6 +5,6 @@ import { ObjectPayload } from './object.payload';
 @Injectable({ providedIn: 'root' })
 export class ObjectService {
  log(object?: ObjectPayload): boolean {
- return object?.isWorking || false;
+ return object?.isWorking ?? false;
  }
 }

packages/ngworker/lumberjack/src/lib/logging/lumberjack-log.builder.ts

@@ -44,7 +44,7 @@ export class LumberjackLogBuilder<TPayload extends LumberjackLogPayload | void =
  message: this.message,
  scope: this.scope,
  createdAt: this.time.getUnixEpochTicks(),
- payload: (payloadArg[0] || this.payload) as Exclude<TPayload, InternalWithStaticPayload>,
+ payload: (payloadArg[0] ?? this.payload) as Exclude<TPayload, InternalWithStaticPayload>,
  };
  }