CODE RUB: Refactor Foundations Test Methods

[KhanbalaRashidov]

Unnecessary codes cleared

[glhays]

Unnecessary codes cleared

@hassanhabib
@KhanbalaRashidov Please explain your reasoning for removing verification calls "Never" from all the tests. Which in turn you reorder the VNO's. please see The-Standard specifically here you can see that these are verifications that no other calls have been made.
https://github.com/hassanhabib/The-Standard/blob/master/2.%20Services/2.1%20Foundations/2.1%20Foundations.md#213112-rules--validations-collector

[KhanbalaRashidov]

Unnecessary codes cleared

@KhanbalaRashidov Please explain your reasoning for removing verification calls "Never" from all the tests. Which in turn you reorder the VNO's
@hassanhabib

Because the VerifyNoOtherCalls() method does the same thing, we don't need to write the Never parts

[glhays]

Unnecessary codes cleared

@KhanbalaRashidov Please explain your reasoning for removing verification calls "Never" from all the tests. Which in turn you reorder the VNO's
@hassanhabib

Because the VerifyNoOtherCalls() method does the same thing, we don't need to write the Never parts

please see The-Standard specifically here you can see that these are verifications that no other calls have been made.
https://github.com/hassanhabib/The-Standard/blob/master/2.%20Services/2.1%20Foundations/2.1%20Foundations.md#213112-rules--validations-collector

[KhanbalaRashidov]

Unnecessary codes cleared

@KhanbalaRashidov Please explain your reasoning for removing verification calls "Never" from all the tests. Which in turn you reorder the VNO's
@hassanhabib

Because the VerifyNoOtherCalls() method does the same thing, we don't need to write the Never parts

please see The-Standard specifically here you can see that these are verifications that no other calls have been made.
https://github.com/hassanhabib/The-Standard/blob/master/2.%20Services/2.1%20Foundations/2.1%20Foundations.md#213112-rules--validations-collector

@glhays It can be written, but it won't do anything. Times.Never because. It's better if we don't write

[KhanbalaRashidov]

Unnecessary codes cleared

@KhanbalaRashidov Please explain your reasoning for removing verification calls "Never" from all the tests. Which in turn you reorder the VNO's
@hassanhabib

Because the VerifyNoOtherCalls() method does the same thing, we don't need to write the Never parts

please see The-Standard specifically here you can see that these are verifications that no other calls have been made.
https://github.com/hassanhabib/The-Standard/blob/master/2.%20Services/2.1%20Foundations/2.1%20Foundations.md#213112-rules--validations-collector

@glhays It can be written, but it won't do anything. Times.Never because. It's better if we don't write

@glhays In the projects we do by applying The Standard, we make the same arrangement as I did, and the result is the same @ElbekDeveloper

[glhays]

Unnecessary codes cleared

@KhanbalaRashidov Please explain your reasoning for removing verification calls "Never" from all the tests. Which in turn you reorder the VNO's
@hassanhabib

Because the VerifyNoOtherCalls() method does the same thing, we don't need to write the Never parts

please see The-Standard specifically here you can see that these are verifications that no other calls have been made.
https://github.com/hassanhabib/The-Standard/blob/master/2.%20Services/2.1%20Foundations/2.1%20Foundations.md#213112-rules--validations-collector

@glhays It can be written, but it won't do anything. Times.Never because. It's better if we don't write

@glhays In the projects we do by applying The Standard, we make the same arrangement as I did, and the result is the same @ElbekDeveloper

yes I understand, I guess my point is that the standard currently calls for it to be written as it currently is. Removing it now may cause others using these projects as learning vesicles, may question why all so different from the published standard. Believe me I am not disputing anything here and I am not the trier of what gets merged was just asking why.

[KhanbalaRashidov]

Unnecessary codes cleared

@KhanbalaRashidov Please explain your reasoning for removing verification calls "Never" from all the tests. Which in turn you reorder the VNO's
@hassanhabib

Because the VerifyNoOtherCalls() method does the same thing, we don't need to write the Never parts

please see The-Standard specifically here you can see that these are verifications that no other calls have been made.
https://github.com/hassanhabib/The-Standard/blob/master/2.%20Services/2.1%20Foundations/2.1%20Foundations.md#213112-rules--validations-collector

@glhays It can be written, but it won't do anything. Times.Never because. It's better if we don't write

@glhays In the projects we do by applying The Standard, we make the same arrangement as I did, and the result is the same @ElbekDeveloper

yes I understand, I guess my point is that the standard currently calls for it to be written as it currently is. Removing it now may cause others using these projects as learning vesicles, may question why all so different from the published standard. Believe me I am not disputing anything here and I am not the trier of what gets merged was just asking why.

@glhays Ustad Hassan brings clarity to this issue. We will understand better

[glhays]

@KhanbalaRashidov FYI see this video for Git verification.
https://youtu.be/u9L5kDlU8rs?si=e0erAk4iD7JK7Zmc

[KhanbalaRashidov]

@KhanbalaRashidov FYI see this video for Git verification. https://youtu.be/u9L5kDlU8rs?si=e0erAk4iD7JK7Zmc

Thanks

[glhays]

@KhanbalaRashidov This specific point 1:20:56 of the following video may enlighten your perspective on the redundancy or uneeded verifications within The Standard testing framework.
https://youtu.be/guJPrIQ0kJk?si=Pqvw8IgLPHvOWpUT