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

Mixin properties are not updated after modifying it via "Modfiy Mixin" menu item. #1372

Closed
btyilmaz opened this issue Jan 4, 2024 · 5 comments · Fixed by #1373
Closed

Mixin properties are not updated after modifying it via "Modfiy Mixin" menu item. #1372

btyilmaz opened this issue Jan 4, 2024 · 5 comments · Fixed by #1373
Assignees
Labels
bug Something isn't working
Milestone

Comments

@btyilmaz
Copy link

btyilmaz commented Jan 4, 2024

.Describe the bug
Mixin properties not updated after modifying it.

Environment:

  • OS: Windows 10
  • OpenTAP Version: 9.22.3

To Reproduce

Steps to reproduce the behavior:

  1. Add a mixin to a test step

  2. Change one of its properties using "Modfiy Mixin" menu item

  3. Observe that mixin's property is not updated after modifying it.

Actual behavior
When a property is changed from the MixinBuilderUI, mixin's property is not updated, See image below:

image

Expected behavior

Property added via a mixin must be updated after modifying the mixin.

Additional context
You can access OpenTAP forum topic from here

@btyilmaz btyilmaz added the bug Something isn't working label Jan 4, 2024
@rmadsen-ks rmadsen-ks self-assigned this Jan 4, 2024
@rmadsen-ks rmadsen-ks added this to the 9.24.0 milestone Jan 4, 2024
@rmadsen-ks
Copy link
Collaborator

I can verify this issue and have added a fix #1373.

Copy link

A fix for this is in OpenTAP version 9.23.0-beta.11+c82b3ad9 or later.

@sebastian-pop
Copy link
Collaborator

I can still reproduce this with OpenTAP 9.23.1-rc.1

mixinsedit

@sebastian-pop sebastian-pop reopened this Jan 17, 2024
@alnlarsen
Copy link
Collaborator

This is an issue with Limit mixin: https://github.com/opentap/basic-mixins/issues/2

@sebastian-pop
Copy link
Collaborator

I can confirm the fix on the Number mixin with OpenTAP 9.23.1-rc.1

Closing this

@alnlarsen alnlarsen modified the milestones: 9.24.0, 9.23.1 Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants