test: add integration test for TypeScript declarations #4853
+113
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ankddev
commented
Aug 14, 2025
lpil
reviewed
Aug 17, 2025
| .PHONY: build | ||
| test: | ||
| cargo run --quiet -- build | ||
| tsc ./build/dev/javascript/typescript_declarations/typescript_declarations.d.mts --noEmit --lib es2015 |
Member
There was a problem hiding this comment.
Can we write this in a way where they don't need to install tsc globally? Perhaps deno can be used?
Contributor
Author
There was a problem hiding this comment.
I tried to use it and it seems that tsc and deno process files in different way so this doesn't work, sadly.
$ deno check ./main.ts
Check file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/main.ts
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'BitArray'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'CustomType'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'Empty'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'Error'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'List'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'NonEmpty'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'Ok'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'Result'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'UtfCodepoint'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'codepointBits'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'divideFloat'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'divideInt'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'isEqual'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'prepend'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'remainderInt'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'sizedFloat'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'sizedInt'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'stringBits'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'toBitArray'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
TS2308 [ERROR]: Module "../prelude.mjs" has already exported a member named 'toList'. Consider explicitly re-exporting to resolve the ambiguity.
export type * from "../prelude.d.mts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
at file:///data/data/com.termux/files/home/gleam/test/typescript_declarations/build/dev/javascript/typescript_declarations/gleam.d.mts:2:1
Found 20 errors.
error: Type checking failed.
Member
There was a problem hiding this comment.
Sorry, what do you mean? Deno isn't used in this test?
Contributor
Author
There was a problem hiding this comment.
@lpil I didn't commit this yet, but tested it locally. I can commit it then if it makes sense
Contributor
Author
|
@lpil I fixed all things you mentioned. Niw test includes more code and there is TS file which imports generated code and type-checking works. One thing that I don't know what to do with is tsc requirement. I tried to use Deno and found following:
|
lpil
reviewed
Dec 4, 2025
Signed-off-by: Andrey <[email protected]>
Signed-off-by: Andrey <[email protected]>
Signed-off-by: Andrey <[email protected]>
Signed-off-by: Andrey <[email protected]>
Signed-off-by: Andrey <[email protected]>
Signed-off-by: Andrey <[email protected]>
Signed-off-by: Andrey <[email protected]>
Contributor
Author
|
@lpil Hello!
Nothing at all! I rewrote th entire test, I belive that now it satisfies requirements. Also, I tried to experiment with different commands:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #2003
Description
Add integration test for TS type declarations. It compiles project and then runs
tscwith--noEmitflag, so it won't generate JS files.