Fix getChildCount/At methods in EndOfFileTokens

[tjjfvi]

Before, they were hardcoded to return 0 and undefined!, respectively, but that is inaccurate for EndOfFileTokens with attached jsdoc.

Fixes #44990

[AviVahl]

Should return Node | undefined?
TypeScript's codebase chould use the stricter tsconfig flags...

[tjjfvi]

Theoretically, yes, but that's somewhat tangential to this PR

[DanielRosenwasser]

Is there a way to get a quick test added?

[tjjfvi]

I added a test to src/testRunner/unittests/publicApi.ts, as that seemed to be the most applicable place. All of the tests are passing, but afaict the "77248 passing" number did not go up, so I'm not sure if there's something else I need to do to add the test.

[DanielRosenwasser]

but afaict the "77248 passing" number did not go up

You can try making the test fail locally as a sanity check which I'd appreciate - maybe each unittest only gets counted as its own test?

[tjjfvi]

Good point -- the test failed successfully.

[sandersn]

You can try making the test fail locally as a sanity check which I'd appreciate - maybe each unittest only gets counted as its own test?

I'm pretty sure unit tests are not part of the 77248 total. They may also run on each process for a parallel test run.