rthomas320 commented on code in PR #1197: URL: https://github.com/apache/daffodil-vscode/pull/1197#discussion_r1991622283
########## src/tests/DfdlIintellisenseTestingChecklist.md: ########## @@ -0,0 +1,145 @@ +<!-- + Licensed to the Apache Software Foundation (ASF) under one or more + contributor license agreements. See the NOTICE file distributed with + this work for additional information regarding copyright ownership. + The ASF licenses this file to You under the Apache License, Version 2.0 + (the "License"); you may not use this file except in compliance with + the License. You may obtain a copy of the License at + + http://www.apache.org/licenses/LICENSE-2.0 + + Unless required by applicable law or agreed to in writing, software + distributed under the License is distributed on an "AS IS" BASIS, + WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + See the License for the specific language governing permissions and + limitations under the License. +--> + +# The following tests use the file: src/tests/data/testDfdlMixedLineFormats.dfdl.xsd + +Place the cursor at line 11 column 44, after alignmentUnits="bytes" press the backspace key until alignmentUnits="bytes" is erased. Type CTRL+space. Select dfdl:alignmentUnits from the dropdown and bytes from the value dropdown. +\[ \] Verify that alignmentUnits="bytes" is inserted. +\[ \] Verify that alignmentUnits does not have the dfdl: prefix. + +Place the cursor on line 28, column 28 at the end of the line \<xs:group +name = \"test\"\>. Type a space. +\[ \] Verify drop down contains xs:annotation and xs:sequence. + +Place the cursor at the same position as above. Type CTRL+space +\[ \] Verify drop down contains xs:annotation and xs:sequence. + +Place the cursor at line 30, column 18 at the end of the line +\<xs:sequence\>. Type a space or CTRL+space. +\[ \] Verify dropdown contains xs:annotation, xs:choice, xs:element +name, xs:element ref, and xs:sequence. + +Place cursor at line 38, column 39 at the end of the line \<xs:element +name=\"Keyword\"\>. Type a space. +\[ \] Verify dropdown contains dfdl:alignment, dfdl:alignmentUnits, +dfdl:binaryBooleanFalseRep, dfdl:binaryBooleanTrueRep, +dfdl:binaryNumberRep, dfdl:bitOrder, dfdl:byteOrder, +dfdl:choiceBranchKey, dfdl:encoding, dfdl:inputValueCalc, dfdl:lenght, +dfdl:lengthKind, dfdl:lengthPattern, dfdl:lengthUnits, dfdl:occursCount, +dfdl:occursCountKind, dfdl:outputNewLine dfdl:outputValueCalc, +dfdl:prefxIncludesPrefixLength, dfdl:prefixLengthType, +dfdl:representation, dfdl:terminator, maxOccurs, minOccurs, name: ref, +and type. + +\[ \] Verify dropdown appears with above listed elements, and the first +line in dropdown is highlighted with a description of the highlighted +item beside it. + +\[ \] Verify a dropdown appears with the above listed elements and +pressing the down arrow key displays the description of each highlighted +element. + +Place the cursor at line 39 and hover over the attribute dfdl:alignment. +\[ \] Verify a text box appears with a definition for dfdl:alignment. + +Place the cursor at line 39 column 47, with the double quotes with the value 1 for dfdl:alignment. +Type CTRL+space. +\[ \] Verify the dropdown contains 1, 2, and implicit. + +Place the cursor at line 39 column 65, after dfdl:length="10". Type CTRL+space. Select dfdl:byteOrder from the dropdown and bigEndian for the value. +\[ \] Verify that dfdl:byteOrder="bigEndian" is inserted and appropriately spaced. Review Comment: It always adds a space. I think the number of spaces depends on how many spaces were there prior to inserting the attribute. A change could possibly be made to check the spaces, but backspacing over existing spaces is complicated. Any requested change would have to be in a later version. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
