Remove transitive mapping tests? · Issue #13 · tc39/source-map-tests · GitHub
Skip to content

Remove transitive mapping tests? #13

Open

Description

@takikawa

In the source map hackfest, we discussed that at least browser consumers are unlikely to want to implement transitive mapping (mapping in which source maps map to original sources that themselves have mappings).

We currently do have tests for this case, but maybe we should remove them?

Activity

  1. nicolo-ribaudo commented on Jun 25, 2024

    @nicolo-ribaudo
    Member

    I think it's valuable to test tools that perform source map composition, and even though the current transitive mappings test is not optimal it maybe helps with that?

  2. dcrousso commented on Oct 8, 2024

    @dcrousso
    Member

    btw im pretty sure WebKit doesn't support this

    it would likely require Web Inspector to actually parse the sourceContent in order to pull out the source map directive/URL instead of just displaying it

    unless this is something super commonly used, id be in favor of removing it and not supporting it as that's a nontrivial amount of work to implement

  3. littledan commented on Jan 8, 2025

    @littledan
    Member

    I think we should include tests on things like source map composition which are useful for tools, and just mark them with some metadata to explain to browsers that the tests aren't for them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions