[test] Use @pytest.mark.forked to fix test_without_init

[archibate]

Related issue = close #1406 replace #1410

[Click here for the format server]

---

@xumingkuan I finally found a systematic solution for this while searching doc for #1462, could you try this out?

[xumingkuan]

It passed! Thank you very much.

[archibate]

Why AppVeyor failed test...

Traceback (most recent call last):
  File "C:\Users\appveyor\AppData\Roaming\Python\Python36\site-packages\py\_process\forkedfunc.py", line 110, in __del__
    if self.pid is not None:  # only clean up in main process
AttributeError: 'ForkedFunc' object has no attribute 'pid'

Did you meet this problem when running ti test or ti test runtime?

[xumingkuan]

Why AppVeyor failed test...

Traceback (most recent call last):
  File "C:\Users\appveyor\AppData\Roaming\Python\Python36\site-packages\py\_process\forkedfunc.py", line 110, in __del__
    if self.pid is not None:  # only clean up in main process
AttributeError: 'ForkedFunc' object has no attribute 'pid'

Did you meet this problem when running ti test or ti test runtime?

Oh, yes I did. But the test passed while showing this message when running either ti test or ti test runtime.

[archibate]

Why AppVeyor failed test...

Traceback (most recent call last):
  File "C:\Users\appveyor\AppData\Roaming\Python\Python36\site-packages\py\_process\forkedfunc.py", line 110, in __del__
    if self.pid is not None:  # only clean up in main process
AttributeError: 'ForkedFunc' object has no attribute 'pid'

Did you meet this problem when running ti test or ti test runtime?

Oh, yes I did. But the test passed while showing this message when running either ti test or ti test runtime.

So does it return a non-zero exit code?

[xumingkuan]

pytest-dev/pytest-forked#6

Looks like this is not solvable with pytest?

[archibate]

Does the latest commit (xfail) works for you?

[xumingkuan]

Does the latest commit (xfail) works for you?

The message is the same (AttributeError: 'ForkedFunc' object has no attribute 'pid').

[archibate]

Are you sure? f64c864? Fine, let me just skip this test on win.
Done, e8e9c8e, could you try this out?

[xumingkuan]

Are you sure? f64c864? Fine, let me just skip this test on win.

Oh, this commit looks not to belong to this PR. I tried it but still AttributeError: 'ForkedFunc' object has no attribute 'pid'.

[xumingkuan]

Are you sure? f64c864? Fine, let me just skip this test on win.
Done, e8e9c8e, could you try this out?

No, it doesn't work either. However, (of course) it works if I comment out @pytest.mark.forked...

[archibate]

Fine, let's just move this test to misc/...

[xumingkuan]

Thanks!

[codecov]

Codecov Report

Merging #1464 into master will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1464      +/-   ##
==========================================
+ Coverage   66.88%   66.92%   +0.03%     
==========================================
  Files          38       38              
  Lines        5433     5421      -12     
  Branches      956      955       -1     
==========================================
- Hits         3634     3628       -6     
+ Misses       1636     1629       -7     
- Partials      163      164       +1     

Impacted Files	Coverage Δ
python/taichi/lang/__init__.py	78.60% <ø> (-0.06%)	⬇️
python/taichi/lang/ops.py	93.12% <ø> (+0.27%)	⬆️
python/taichi/lang/util.py	64.70% <ø> (-0.21%)	⬇️
python/taichi/misc/error.py	14.28% <ø> (ø)
python/taichi/__init__.py	92.00% <100.00%> (+0.33%)	⬆️
python/taichi/lang/shell.py	40.67% <0.00%> (-3.39%)	⬇️
python/taichi/lang/ast_checker.py	74.28% <0.00%> (+2.06%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a63714c...63d7914. Read the comment docs.