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

fo-test-fn-graphemes-004 #161

Open
ndw opened this issue Sep 13, 2024 · 1 comment
Open

fo-test-fn-graphemes-004 #161

ndw opened this issue Sep 13, 2024 · 1 comment

Comments

@ndw
Copy link
Contributor

ndw commented Sep 13, 2024

@michaelhkay this generated test is broken:

  <test-case name="fo-test-fn-graphemes-004">
      <description>Test case for fn:graphemes</description>
      <created by="Michael Kay using generate-qt3-test-set.xsl" on="2024-09-13"/>
      <environment ref="global"/>
      <test>
          let $crlf := 
          char('\r') || char('\n')
           return 

        
        graphemes($crlf)
      </test>
      <result>
         <assert-deep-eq>$crlf</assert-deep-eq>
      </result>
   </test-case>

I'm not sure if there's already a mechanism to suppress some examples from generating tests, or if you think it's worth fixing the way the example is written.

@ChristianGruen
Copy link
Contributor

ChristianGruen commented Sep 13, 2024

The test will be changed by qt4cg/qtspecs#1434, but we could also drop it completely. It needs to be read at least twice to understand what it wants to express.

What do you think?

See also qt4cg/qtspecs#1373

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

No branches or pull requests

2 participants