Checking for jid before returning data

[brejoc]

What does this PR do?

Seems raw can have returns for the same minion, but an other job. In order to not return resutls from the wrong job, we need to check for the jid.

What issues does this PR fix or reference?

#50927

Previous Behavior

Wrong job results might be returned. Please take a look at #50927

New Behavior

Only the correct jobs results should be returned.

Tests written?

No

Commits signed with GPG?

Yes

[Ch3LL]

Can you write a test for this change?

[brejoc]

@Ch3LL Sorry for dragging my feet here for so long, but finally I've just added two test cases. Could you please check your the test runs? Hudson seems to have problems fetching the changes.

[Ch3LL]

I restarted the tests. I went ahead and tested this PR against my test case i pasted in the issue and unfortunately im still seeing the issue:

(salt-py2) ➜  ~ for i in {1..100}; do sudo python2 ~/programa.py; done
{u'newrecipe': {u'ret': u'testa'}}
{u'newrecipe': {u'ret': u'testa'}}
{u'newrecipe': {u'ret': u'testa'}}
{u'newrecipe': {u'ret': u'testa'}}
{u'newrecipe': {u'ret': u'testb'}}
{u'newrecipe': {u'ret': u'testa'}}

are you seeing it fixed for you? I tried multiple times to make sure.

[brejoc]

@Ch3LL I've been re-running the tests several times and for me the issue is gone. As soon as I remove the patch, I see it again. Are you running both scripts on the master?

[brejoc]

@Ch3LL I'm closing this PR because I've just opened the same PR against the master branch: #55216