testsuites

tst_CANBusFaultCount

Clone Tools

Stats

Commits this week: 0
Total Committers: 5
Last Commit: 20 Nov
Commits this week: 0

Lines of code count not available

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Wednesday 21 Feb 2024
5:43 pm

Behrouz NematiPour

12:36 pm

Behrouz NematiPour

Thursday 07 Dec 2023
11:51 am

msuleiman  (deleted user)

Monday 04 Dec 2023
10:47 am

jtaylor  (deleted user)

Friday 03 Nov 2023
11:30 am

jtaylor  (deleted user)

9:12 am

Dara Navaei

DEN-16533-UI-VD-S109 - Integration Update 2
DEN-16533-UI-VD-S109 - Integration Update 2
Monday 15 Feb 2021
1:28 pm

plucia  (deleted user)

1:27 pm

Behrouz NematiPour

10:25 am

Behrouz NematiPour

Still don't feel comfortable with this solution and haven't been convinced that it should be the way to get around the SquishQt issue of finding the object within the given timeout. On the other ha...

Still don't feel comfortable with this solution and haven't been convinced that it should be the way to get around the SquishQt issue of finding the object within the given timeout.
On the other hand, don't have a better idea to stop the test from failing randomly.
So would be better to keep it for now.

Wednesday 27 Jan 2021
10:10 am

Behrouz NematiPour

RESOLVED

RESOLVED

10:05 am

Behrouz NematiPour

Please change to TreatmentStates RESOLVED

Please change to TreatmentStates
RESOLVED

9:54 am

plucia  (deleted user)

Nothing has changed about the usage of the SquishQt API, other than the fact that the dictionary passed to waitForObjectExists previously wasn't really testing anything. Removing utils.dict_update ...

Nothing has changed about the usage of the SquishQt API, other than the fact that the dictionary passed to waitForObjectExists previously wasn't really testing anything.
Removing utils.dict_update it will break the test case.

Tuesday 26 Jan 2021
2:54 pm

plucia  (deleted user)

I will discuss with Peman and get back to you.

I will discuss with Peman and get back to you.

2:08 pm

Behrouz NematiPour

I believe we discussed enough. And it was discussed verbally before to not to do it. Nothing has changed about the usage of the SquishQt API. We pass a dictionary to waitForObjectExists prior to ...

I believe we discussed enough.
And it was discussed verbally before to not to do it.


Nothing has changed about the usage of the SquishQt API. We pass a dictionary to waitForObjectExists prior to this change and after.
Removing it will break the test case.

If nothing has changed so why your test would break.
Please change it to the recommended ways in SquishQt examples.

1:36 pm

Behrouz NematiPour

The way it was used is in SquishQt examples, but I can't find any example that SquishQt is being used the way you used it. Please change it to the recommended ways in SquishQt examples.

The way it was used is in SquishQt examples, but I can't find any example that SquishQt is being used the way you used it.
Please change it to the recommended ways in SquishQt examples.

1:25 pm

plucia  (deleted user)

I think you've misunderstood. The bug is not part of the SquishQt API. It is a problem with the way the SquishQt API is being used.

I think you've misunderstood. The bug is not part of the SquishQt API. It is a problem with the way the SquishQt API is being used.

1:02 pm

Behrouz NematiPour

If there is a bug in SquishQt please report to Froglogic and give feedback and please don't try to fix it by yourself. In case they fix the bug (if there is a bug) and they fix it, and the fix is n...

If there is a bug in SquishQt please report to Froglogic and give feedback and please don't try to fix it by yourself.
In case they fix the bug (if there is a bug) and they fix it, and the fix is not compatible with your fix, then we have self-generated bugs.
Please remove these and let Froglogic know you found a bug and let them help us.

12:59 pm

Behrouz NematiPour

RESOLVED

RESOLVED

11:23 am

plucia  (deleted user)

We are in agreement that each test should be self contained. To ensure each test is self-contained, each test needs to have a separate instance of HDSimulator. The design of dialin is not the focus...

We are in agreement that each test should be self contained.
To ensure each test is self-contained, each test needs to have a separate instance of HDSimulator.
The design of dialin is not the focus of this review...

11:23 am

plucia  (deleted user)

Tx_States violates the CapWords convention for classes as per our coding standard, as no underscores should be present between words in a class name. TreatmentStates is clearer than TxStates. Tx co...

Tx_States violates the CapWords convention for classes as per our coding standard, as no underscores should be present between words in a class name.
TreatmentStates is clearer than TxStates. Tx could easily be confused with transmit...

11:10 am

plucia  (deleted user)

I agree. To ensure I don't miss additional instances of HDSimulator I'll make this change on the latest testsuites branch I'm working on.

I agree. To ensure I don't miss additional instances of HDSimulator I'll make this change on the latest testsuites branch I'm working on.

11:01 am

Behrouz NematiPour

I don't mean to limit the HDSimulator to static functions, it can have static and normal methods. 1 - The test scripts are run in a consecutive manner, not in parallel. Each test should be self c...

I don't mean to limit the HDSimulator to static functions, it can have static and normal methods.


1 - The test scripts are run in a consecutive manner, not in parallel. Each test should be self contained.

I agree in this case but the general implementation and design of HDSimulator lead to that misuse of the class.

10:53 am

Behrouz NematiPour

Please consider changing the instance name to hd_simulator. The name of variables is always important, especially in this case which makes it confusing since the name of the class and the variable ...

Please consider changing the instance name to hd_simulator.
The name of variables is always important, especially in this case which makes it confusing since the name of the class and the variable are the same.

10:51 am

Behrouz NematiPour

It has been defined as a class not a variable after the name change. So the name should be "Tx_States"

It has been defined as a class not a variable after the name change.
So the name should be "Tx_States"

10:49 am

Behrouz NematiPour

RESOLVED

RESOLVED

Monday 25 Jan 2021
10:56 am

plucia  (deleted user)

Limiting the HDSimulator to static functions prevents you from maintaining any state between successive calls and is incompatible with the DenaliCanMessenger. It is an improvement to hold state inf...

Limiting the HDSimulator to static functions prevents you from maintaining any state between successive calls and is incompatible with the DenaliCanMessenger. It is an improvement to hold state information and allows us to use the DenaliCanMessenger. Using the DenaliCanMessenger and dialin eliminates the need to maintain duplicated codebases and allows for bidirectional messaging. Your squish folder was doing the same thing as dialin but on a smaller scale.

1 - The test scripts are run in a consecutive manner, not in parallel. Each test should be self contained.
2 - (See statement above). This comment is about dialin, not testsuites.
3 - (See statement above). This comment is about dialin, not testsuites.

10:54 am

plucia  (deleted user)

Also since the variable name stylings are not considered major issues, I will take a note and make sure to update the variable name at a later time.

Also since the variable name stylings are not considered major issues, I will take a note and make sure to update the variable name at a later time.

10:52 am

plucia  (deleted user)

1 - Since the name of the variable is not a major issue, I have noted your comment and will update the name to hd_simulator in a future revision. 2 - It does, it was added in a future revision. Ple...

1 - Since the name of the variable is not a major issue, I have noted your comment and will update the name to hd_simulator in a future revision.
2 - It does, it was added in a future revision. Please keep in mind this code review is 8 weeks old.

10:51 am

plucia  (deleted user)

I changed it from tx to TX to make it clearer it was a constant, but to maximize readability I think it should be TREATMENT_STATES, or TX_STATES to align with our coding standard, not TxStates. Fr...

I changed it from tx to TX to make it clearer it was a constant, but to maximize readability I think it should be TREATMENT_STATES, or TX_STATES to align with our coding standard, not TxStates.

From our coding standard:


Constants are usually defined on a module level and written in all capital letters with
underscores separating words. Examples include MAX_OVERFLOW and TOTAL

10:51 am

plucia  (deleted user)

At the time, this must have been the latest commit on the dialin master branch. It isn't maintained manually. It's generated automatically when issuing the pip freeze > requirements.txt command.

At the time, this must have been the latest commit on the dialin master branch.
It isn't maintained manually. It's generated automatically when issuing the pip freeze > requirements.txt command.