<!-- 
RSS generated by JIRA (9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66) at Thu Feb 08 08:24:37 UTC 2024

It is possible to restrict the fields that are returned in this document by specifying the 'field' parameter in your request.
For example, to request only the issue key and summary append 'field=key&field=summary' to the URL of your request.
-->
<rss version="0.92" >
<channel>
    <title>MongoDB Jira</title>
    <link>https://jira.mongodb.org</link>
    <description>This file is an XML representation of an issue</description>
    <language>en-us</language>    <build-info>
        <version>9.7.1</version>
        <build-number>970001</build-number>
        <build-date>13-04-2023</build-date>
    </build-info>


<item>
            <title>[DRIVERS-2060] Improve integration tests for SRV polling</title>
                <link>https://jira.mongodb.org/browse/DRIVERS-2060</link>
                <project id="10980" key="DRIVERS">Drivers</project>
                    <description>&lt;p&gt;One of the feedback items from the 4.2 retro was that SRV polling spec tests were incomplete (or lacking) and that the &quot;initial implementation did not have any integration tests, &lt;span class=&quot;error&quot;&gt;&amp;#91;and&amp;#93;&lt;/span&gt; lots of drivers had to make up the tests.&quot; &lt;/p&gt;

&lt;p&gt;I assume this refers to the &lt;a href=&quot;https://github.com/mongodb/specifications/blob/master/source/polling-srv-records-for-mongos-discovery/polling-srv-records-for-mongos-discovery.rst#test-plan&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;Test Plan&lt;/a&gt; in the spec, which dates back to &lt;a href=&quot;https://github.com/mongodb/specifications/commit/49d2c8dda9efb4ef862fc5e21586ce5ead883745&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;49d2c8d&lt;/a&gt; (and old commit much earlier in the 4.2 dev cycle). That section also admits that testing is difficult because it involves modifying DNS records.&lt;/p&gt;

&lt;p&gt;If any drivers were able to implement better integration tests for this feature, we&apos;d do well to contribute those back into the spec so all languages can benefit.&lt;/p&gt;</description>
                <environment></environment>
        <key id="971558">DRIVERS-2060</key>
            <summary>Improve integration tests for SRV polling</summary>
                <type id="14901" iconUrl="https://jira.mongodb.org/secure/viewavatar?size=xsmall&amp;avatarId=14700&amp;avatarType=issuetype">Spec Change</type>
                                            <priority id="3" iconUrl="https://jira.mongodb.org/images/icons/priorities/major.svg">Major - P3</priority>
                        <status id="10038" iconUrl="https://jira.mongodb.org/images/icons/subtask.gif" description="">Backlog</status>
                    <statusCategory id="2" key="new" colorName="default"/>
                                    <resolution id="-1">Unresolved</resolution>
                                        <assignee username="-1">Unassigned</assignee>
                                    <reporter username="jmikola@mongodb.com">Jeremy Mikola</reporter>
                        <labels>
                    </labels>
                <created>Wed, 16 Oct 2019 21:54:22 +0000</created>
                <updated>Thu, 31 Mar 2022 13:43:52 +0000</updated>
                                                                <component>SRV Polling</component>
                                        <votes>0</votes>
                                    <watches>2</watches>
                                                                                                                <comments>
                            <comment id="2487337" author="jmikola@gmail.com" created="Thu, 17 Oct 2019 18:20:06 +0000"  >&lt;blockquote&gt;&lt;p&gt;To make it clear that there are tests, is it OK to move their descriptions to tests/README.md?&lt;/p&gt;&lt;/blockquote&gt;

&lt;p&gt;I think that&apos;s a good idea. I actually didn&apos;t realize there was a &lt;tt&gt;tests/&lt;/tt&gt; directory for the spec and had only seen the prose tests myself. FWIW, PHP didn&apos;t implement these and simply relied on libmongoc&apos;s test coverage.&lt;/p&gt;

&lt;p&gt;I&apos;d propose moving the prose tests into &lt;tt&gt;tests/&lt;/tt&gt; in a distinct PR with no associated SPEC ticket (since it&apos;s not relevant to any driver changes). Then we can consider adding the additional tests you link in SPEC-1407 via this ticket &amp;#8211; unless you think the newly acknowledged prose tests will suffice. &lt;/p&gt;</comment>
                            <comment id="2487209" author="oleg.pudeyev" created="Thu, 17 Oct 2019 17:25:13 +0000"  >&lt;p&gt;There actually are some prose tests in the spec which I may have overlooked because they are specified in the spec document and not in the tests directory. To make it clear that there are tests, is it OK to move their descriptions to tests/README.md?&lt;/p&gt;

&lt;p&gt;This should be done after &lt;a href=&quot;https://github.com/mongodb/specifications/pull/663&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://github.com/mongodb/specifications/pull/663&lt;/a&gt; is merged.&lt;/p&gt;

&lt;p&gt;I added a list of additional tests implemented in Ruby driver in &lt;a href=&quot;https://jira.mongodb.org/browse/SPEC-1407?focusedCommentId=2481322&amp;amp;page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-2481322&quot; class=&quot;external-link&quot; rel=&quot;nofollow&quot;&gt;https://jira.mongodb.org/browse/SPEC-1407?focusedCommentId=2481322&amp;amp;page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-2481322&lt;/a&gt;.&lt;/p&gt;</comment>
                            <comment id="2485836" author="jmikola@gmail.com" created="Wed, 16 Oct 2019 21:54:59 +0000"  >&lt;p&gt;&lt;a href=&quot;https://jira.mongodb.org/secure/ViewProfile.jspa?name=oleg.pudeyev&quot; class=&quot;user-hover&quot; rel=&quot;oleg.pudeyev&quot;&gt;oleg.pudeyev&lt;/a&gt;: I believe this was your retro feedback. Can you provide some more insight on this? Are there additional tests in Ruby that could be contributed upstream today?&lt;/p&gt;</comment>
                    </comments>
                <issuelinks>
                            <issuelinktype id="10011">
                    <name>Depends</name>
                                                                <inwardlinks description="is depended on by">
                                                        </inwardlinks>
                                    </issuelinktype>
                            <issuelinktype id="10012">
                    <name>Related</name>
                                            <outwardlinks description="related to">
                                                        </outwardlinks>
                                                        </issuelinktype>
                    </issuelinks>
                <attachments>
                    </attachments>
                <subtasks>
                    </subtasks>
                <customfields>
                                                                                                                                                                                                                                                                                                            <customfield id="customfield_15850" key="com.atlassian.jira.plugins.jira-development-integration-plugin:devsummary">
                        <customfieldname>Development</customfieldname>
                        <customfieldvalues>
                            
                        </customfieldvalues>
                    </customfield>
                                                                                                                                                    <customfield id="customfield_10951" key="com.atlassian.jira.plugin.system.customfieldtypes:radiobuttons">
                        <customfieldname>Driver Changes</customfieldname>
                        <customfieldvalues>
                                <customfieldvalue key="10748"><![CDATA[Needed]]></customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                        <customfield id="customfield_12550" key="com.pyxis.greenhopper.jira:gh-lexo-rank">
                        <customfieldname>Rank</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>2|hrbx4n:</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                <customfield id="customfield_10558" key="com.pyxis.greenhopper.jira:gh-global-rank">
                        <customfieldname>Rank (Obsolete)</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>9223372036854775807</customfieldvalue>
                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                        </customfields>
    </item>
</channel>
</rss>