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

Issue # 94 - Added a new binding and unit test to assert if the auto-… #161

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rashish12
Copy link

@rashish12 rashish12 commented Nov 15, 2022

A new entity step binding has been added to assert if the auto-number field contains value as per the input pattern/format those defined by the Power Platform Docs.
This resolves the issue #94

Purpose

To validate the format of an auto-number field when a record is created

Approach

Added a new entity step binding to assert the auto-number field value_

TODOs

  • Automated test coverage for new code
  • Documentation updated (if required)
  • Build and tests successful

…number field contains value as per the input pattern/format
Copy link
Contributor

@tdashworth tdashworth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rashish12 Thanks for the contribution!

There are some failing tests but on review, they seem to be unrelated.

@tdashworth
Copy link
Contributor

@ewingjm what are your thoughts on adding a "ignore tests result" flag to the pipeline so that it doesn't fail? By default it would be false, of course but if we choose to re-run it, we could override it.

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.

2 participants