tiagobento commented on PR #2456:
URL: 
https://github.com/apache/incubator-kie-tools/pull/2456#issuecomment-2245712046

   @ljmotta The idea of not having variables was to prevent tests from becoming 
complicated. We wanted to have testes that could be read from start to finish 
without having to jump around too much to understand what the test case is. I 
see on this PR we have some variables already, but I think I'd go for a single 
variable at the top and then inlining stuff at the individual lines.. Example:
   
   ```typescript
         const decisionTable = bee.expression.asDecisionTable()
         await decisionTable.cellAt({ row: 1, column: 1 }).fill("test");
         await decisionTable.cellAt({ row: 1, column: 0 }).contextMenu.open();
         await decisionTable.cellAt({ row: 1, column: 0 
}).contextMenu.option("Insert").click();
         await decisionTable.cellAt({ row: 1, column: 0 
}).contextMenu.button("minus").click();
         await decisionTable.cellAt({ row: 1, column: 0 
}).contextMenu.radio("Below").click();
         await decisionTable.cellAt({ row: 1, column: 0 
}).contextMenu.button("Insert").click();
   
         await expect(decisionTable.cellAt({ row: 1, column: 1 
}).content).toContainText("test");
         await expect(decisionTable.cellAt({ row: 2, column: 0 
}).content).toBeAttached();
   ```


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to