import unittest
from astropy.tests.helper import remote_data
from astroquery.dace import Dace

HARPS_PUBLICATION = '2009A&A...493..639M'


@remote_data
class TestDaceClass(unittest.TestCase):

    def test_should_get_radial_velocities(self):
        radial_velocities_table = Dace.query_radial_velocities('HD40307')
        assert radial_velocities_table is not None and 'rv' in radial_velocities_table.colnames
        assert 'HARPS' in radial_velocities_table['ins_name']
        assert HARPS_PUBLICATION in radial_velocities_table['pub_bibcode']
        public_harps_data = [row for row in radial_velocities_table['pub_bibcode'] if HARPS_PUBLICATION in row]
        assert len(public_harps_data) > 100


if __name__ == "__main__":
    unittest.main()

import unittest
from astropy.tests.helper import remote_data
from astroquery.dace import Dace

HARPS_PUBLICATION = '2009A&A...493..639M'


@remote_data
class TestDaceClass(unittest.TestCase):

    def test_should_get_radial_velocities(self):
        radial_velocities_table = Dace.query_radial_velocities('HD40307')
        assert radial_velocities_table is not None and 'rv' in radial_velocities_table.colnames
        # HARPS is a spectrograph and has to be present for this target because HD40307 has been observed and
        # processed by this instrument
        assert 'HARPS' in radial_velocities_table['ins_name']
        assert HARPS_PUBLICATION in radial_velocities_table['pub_bibcode']
        public_harps_data = [row for row in radial_velocities_table['pub_bibcode'] if HARPS_PUBLICATION in row]
        assert len(public_harps_data) > 100


if __name__ == "__main__":
    unittest.main()

Add comment to explain the test with HARPS instrument
Add comment to explain the test with HARPS instrument

  import unittest
  from astropy.tests.helper import remote_data
  from astroquery.dace import Dace
  
  HARPS_PUBLICATION = '2009A&A...493..639M'
  
  
  @remote_data
  class TestDaceClass(unittest.TestCase):
  
      def test_should_get_radial_velocities(self):
          radial_velocities_table = Dace.query_radial_velocities('HD40307')
          assert radial_velocities_table is not None and 'rv' in radial_velocities_table.colnames
+         # HARPS is a spectrograph and has to be present for this target because HD40307 has been observed and
+         # processed by this instrument
          assert 'HARPS' in radial_velocities_table['ins_name']
          assert HARPS_PUBLICATION in radial_velocities_table['pub_bibcode']
          public_harps_data = [row for row in radial_velocities_table['pub_bibcode'] if HARPS_PUBLICATION in row]
          assert len(public_harps_data) > 100
  
  
  if __name__ == "__main__":
      unittest.main()
  
Add comment to explain the test with HARPS instrument
## Code Before:
import unittest
from astropy.tests.helper import remote_data
from astroquery.dace import Dace

HARPS_PUBLICATION = '2009A&A...493..639M'


@remote_data
class TestDaceClass(unittest.TestCase):

    def test_should_get_radial_velocities(self):
        radial_velocities_table = Dace.query_radial_velocities('HD40307')
        assert radial_velocities_table is not None and 'rv' in radial_velocities_table.colnames
        assert 'HARPS' in radial_velocities_table['ins_name']
        assert HARPS_PUBLICATION in radial_velocities_table['pub_bibcode']
        public_harps_data = [row for row in radial_velocities_table['pub_bibcode'] if HARPS_PUBLICATION in row]
        assert len(public_harps_data) > 100


if __name__ == "__main__":
    unittest.main()

## Instruction:
Add comment to explain the test with HARPS instrument
## Code After:
import unittest
from astropy.tests.helper import remote_data
from astroquery.dace import Dace

HARPS_PUBLICATION = '2009A&A...493..639M'


@remote_data
class TestDaceClass(unittest.TestCase):

    def test_should_get_radial_velocities(self):
        radial_velocities_table = Dace.query_radial_velocities('HD40307')
        assert radial_velocities_table is not None and 'rv' in radial_velocities_table.colnames
        # HARPS is a spectrograph and has to be present for this target because HD40307 has been observed and
        # processed by this instrument
        assert 'HARPS' in radial_velocities_table['ins_name']
        assert HARPS_PUBLICATION in radial_velocities_table['pub_bibcode']
        public_harps_data = [row for row in radial_velocities_table['pub_bibcode'] if HARPS_PUBLICATION in row]
        assert len(public_harps_data) > 100


if __name__ == "__main__":
    unittest.main()
