Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add (failing) test case to repro #407 #817

Merged
merged 1 commit into from
May 20, 2024
Merged

Conversation

anthrotype
Copy link
Member

will be fixed by #779

I push it separately to demostrate this

@anthrotype
Copy link
Member Author

anthrotype commented May 20, 2024

ttx -q -o - -t GDEF "build-broken/font.ttf" 2>/dev/null
ttx -q -o - -t GDEF "build-fixed/font.ttf" 2>/dev/null
--- /dev/fd/63  2024-05-20 13:44:20
+++ /dev/fd/62  2024-05-20 13:44:20
@@ -15,7 +15,7 @@
             <EndCoord value="0.0"/>
           </VarRegionAxis>
           <VarRegionAxis index="1">
-            <StartCoord value="0.6"/>
+            <StartCoord value="0.0"/>
             <PeakCoord value="1.0"/>
             <EndCoord value="1.0"/>
           </VarRegionAxis>

fails as expected:

---- tests::intermediate_layer_in_designspace stdout ----
thread 'tests::intermediate_layer_in_designspace' panicked at fontc/src/lib.rs:2124:9:
assertion failed: `(left == right)`
Diff < left / right > :
 [
     [
         0.0,
         0.0,
         0.0,
     ],
     [
<        0.5999756,
>        0.0,
         1.0,
         1.0,
     ],
 ]

@anthrotype anthrotype added this pull request to the merge queue May 20, 2024
Merged via the queue into main with commit df157e5 May 20, 2024
9 of 10 checks passed
@anthrotype anthrotype deleted the failing-test-case-for-407 branch May 20, 2024 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant