Accept ToS: fix

[bidlocode]

Emits a warning when run fresh without datadir and asks to accept ToS one more time after that. ToS acceptance procedure happens before anything else so datadir should be created.

[terencechain]

Can we please add a test?

I wonder if we should put this check in the DB pkg so we don't have to create twice

[codecov]

Codecov Report

Merging #7539 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7539      +/-   ##
==========================================
- Coverage   61.83%   61.78%   -0.05%     
==========================================
  Files         422      422              
  Lines       29714    29459     -255     
==========================================
- Hits        18374    18202     -172     
+ Misses       8394     8340      -54     
+ Partials     2946     2917      -29     

[bidlocode]

Modified test so it will test directory creation too. No need to create twice. In docs it says that os.MkdirAll() doesn't do anything if directory already exists.

[prestonvanloon]

@terencechain there are tests already. If we ran these tests on CI for windows, it would have failed, but we only run linux agents at the moment.