<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 9, 2017 at 8:55 AM, Miroslava Voglova <span dir="ltr"><<a href="mailto:mvoglova@redhat.com" target="_blank">mvoglova@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail-HOEnZb"><div class="gmail-h5"><div dir="ltr"><div class="gmail_quote"><div dir="ltr">Hi,<div><br></div><div>I am currently moving all option values from ConfigValues to database. It should make this values more readable and less error prone, since all will be in one place. Also it will allow engine-config to work with all available options and provide a way to find out if option is versioned, or has one general value.</div><div><br></div><div>But I encountered problem with unit tests. Some tests (tests that use MockConfigRule e.g. NetworkInSyncWithVdsNetwo<wbr>rkInterfaceTest) rely on getting default value from ConfigUtilBase#getValue, which will no longer be possible, because there will be no @DefaultValueAttribute. As I understand, this tests cannot get values from database either.</div></div></div></div></div></div></blockquote><div><br><div style="font-family:arial,helvetica,sans-serif" class="gmail_default">Looking at code, it seems that ConfigUtilsBase makes the assumption that
default value for given VdcOption [1] is available through annotations,
driven by the presence of @TypeConverterAttribute + @DefaultValueAttribute. If we want to remove those annotations throughout ConfigValues, we could consider adapting ConfigUtilsBase accordingly.<br></div><br><div style="font-family:arial,helvetica,sans-serif" class="gmail_default">[1] ConfigUtilsBase#getValue(VdcOption)</div><br><div style="font-family:arial,helvetica,sans-serif" class="gmail_default">Or, simply allow the test to specify the given VdcOption's default value explicitly (avoid relying on annotation-driven default value resolution) which can reduce the amount of "magic" in the test :-)<br></div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail-HOEnZb"><div class="gmail-h5"><div dir="ltr"><div class="gmail_quote"><div dir="ltr"><div><br></div><div>Does someone have an idea how to solve this problem? I was thinking about changing MockConfigRule to get values from 000_config.sql, but I am not sure if this change won't cause some other issues. </div><div><br></div><div>Thanks,</div><div>Mirka</div></div>
</div><br></div>
</div></div><br>______________________________<wbr>_________________<br>
Devel mailing list<br>
<a href="mailto:Devel@ovirt.org">Devel@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/<wbr>mailman/listinfo/devel</a><br></blockquote></div><br></div></div>